| Age | Commit message (Collapse) | Author | |
|---|---|---|---|
| 2016-12-16 | [IR] Remove the DIExpression field from DIGlobalVariable. | Adrian Prantl | |
| This patch implements PR31013 by introducing a DIGlobalVariableExpression that holds a pair of DIGlobalVariable and DIExpression. Currently, DIGlobalVariables holds a DIExpression. This is not the best way to model this: (1) The DIGlobalVariable should describe the source level variable, not how to get to its location. (2) It makes it unsafe/hard to update the expressions when we call replaceExpression on the DIGLobalVariable. (3) It makes it impossible to represent a global variable that is in more than one location (e.g., a variable with multiple DW_OP_LLVM_fragment-s). We also moved away from attaching the DIExpression to DILocalVariable for the same reasons. <rdar://problem/29250149> https://llvm.org/bugs/show_bug.cgi?id=31013 Differential Revision: https://reviews.llvm.org/D26769 llvm-svn: 289902 | |||
| 2016-12-12 | Fix MSVC build after 289461; MSVC isn't sure if this is std:: or llvm:: | Nico Weber | |
| llvm-svn: 289480 | |||
| 2016-12-12 | Refactor BitcodeReader: move Metadata and ValueId handling in their own ↵ | Mehdi Amini | |
| class/file Summary: I'm planning on changing the way we load metadata to enable laziness. I'm getting lost in this gigantic files, and gigantic class that is the bitcode reader. This is a first toward splitting it in a few coarse components that are more easily understandable. Reviewers: pcc, tejohnson Subscribers: mgorny, llvm-commits, dexonsmith Differential Revision: https://reviews.llvm.org/D27646 llvm-svn: 289461 | |||
