This patch schedules the pass to add discriminators.
It depends on http://llvm-reviews.chandlerc.com/D2691.
Details
- Reviewers
echristo
Diff Detail
Event Timeline
Update call to DIBuilder::createLexicalBlock.
There is a new argument for discriminators.
lib/CodeGen/CGDebugInfo.cpp | ||
---|---|---|
2581 | This change doesn't seem relevant to this patch? |
This adds two additional changes required by the discriminator
pass:
1- Add discriminator argument when calling DIBuilder::createLexicalBlock.
2- Modify line table test to account for new lexical block created to
support the new discriminator value.
Comments inline.
lib/CodeGen/CGDebugInfo.cpp | ||
---|---|---|
2582 | This can go in before, it just means that nothing will deal with it yes? In addition, it would be good to test that nothing changes if you pass 0 here and don't actually run the pass. | |
test/CodeGenCXX/linetable-eh.cpp | ||
9 ↗ | (On Diff #7183) | This is with the pass running? |
This change doesn't seem relevant to this patch?