Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Reinstate r284008 reverted in r284081, with two fixes: | Richard Smith | 2016-10-14 | 1 | -0/+5 |
| | | | | | | | | | | | | | | | | | | | 1) Merge and demote variable definitions when we find a redefinition in MergeVarDecls, not only when we find one in AddInitializerToDecl (we only reach the second case if it's the addition of the initializer itself that converts an existing declaration into a definition). 2) When rebuilding a redeclaration chain for a variable, if we merge two definitions together, mark the definitions as merged so the retained definition is made visible whenever the demoted definition would have been. Original commit message (from r283882): [modules] PR28752: Do not instantiate variable declarations which are not visible. Original patch by Vassil Vassilev! Changes listed above are mine. llvm-svn: 284284 | ||||
* | Revert r284008. This is us to fail to instantiate static data members in some | Richard Smith | 2016-10-12 | 1 | -5/+0 |
| | | | | | | cases. I'm working on reducing a testcase. llvm-svn: 284081 | ||||
* | Reinstate r283887 and r283882. | Vassil Vassilev | 2016-10-12 | 1 | -0/+5 |
| | | | | | | | | | | | Original message: "[modules] PR28752: Do not instantiate variable declarations which are not visible. https://reviews.llvm.org/D24508 Patch developed in collaboration with Richard Smith!" llvm-svn: 284008 | ||||
* | Revert r283887 and r283882, until the issue is understood and fixed. | Vassil Vassilev | 2016-10-11 | 1 | -5/+0 |
| | | | | llvm-svn: 283890 | ||||
* | [modules] PR28752: Do not instantiate variable declarations which are not ↵ | Vassil Vassilev | 2016-10-11 | 1 | -0/+5 |
visible. https://reviews.llvm.org/D24508 Patch developed in collaboration with Richard Smith! llvm-svn: 283882 |