User Details
- User Since
- Feb 1 2018, 11:54 PM (225 w, 10 h)
Wed, May 25
You, likely, not need D113743 (it'd been reverted just after being committed, so it seems it couldn't be the guilty one).
Mostly. The alternative implementation reuses large part of work of this patch.
Basically, this first implementation has (had) 3 patches in sequence:
Mon, May 23
Thank you, @asavonic for your comments!
Fix comment.
Tue, May 17
Mon, May 16
There is an alternative implementation that relies on addition field of DISubprogram/DILexicalScope that tracks static locals, local imports and types D125693.
Mark this and D125691 as WIP since I'm still testing the approach on various combinations of debug info and optimization options (O0, O3, thinlto, split-dwarf, split-dwarf-inlining, gline-tables-only, etc.).
Except of the testing purpose itself I'm also trying to catch the issue reported by David here https://reviews.llvm.org/D113741#3437182 for previous implementation (while I hope this approach wouldn't trigger the issue, it'd be good to catch and fix it anyway).
Abandon for now.
Issue on gitlab: https://github.com/llvm/llvm-project/issues/54475
Rebase on ToT.
Sat, May 14
Fri, May 13
Apr 8 2022
Mar 30 2022
Mar 21 2022
Mar 14 2022
Friendly ping :)
Mar 7 2022
Ping
Ping
Mar 1 2022
@tra thank you for looking at this!
Feb 28 2022
Apply David's comment and add 'REQUIRES: object-emission' to tests.
Feb 25 2022
Fix a test.
Add SM_70 requirement for 'match' builtins.
Feb 24 2022
Attempt 2.
Minor fix.
Rebase on top of 'main'.
@dblaikie I've updated the patch with a fix for split dwarf issue (the test is included). Could I ask you to check the full build on your side?
- Rebase on ToT & minor fixes.
- Fix the issue when an local imported entity caused emission of multiple CUs in a dwo when split dwarf is enabled.
- Add a test for the aforementioned issue.
Rebase & align checks with new order in the following tests:
Feb 23 2022
Feb 21 2022
Ping
Feb 2 2022
Dec 28 2021
Dec 23 2021
Dec 22 2021
@dblaikie, thank you for the reproducers!
Dec 15 2021
Dec 10 2021
@rtrieu, thank you for looking at this! Do you have any comments about the rest of the patch? Do you think it makes sense?
Dec 8 2021
Dec 7 2021
Dec 6 2021
Dec 5 2021
Dec 4 2021
Dec 3 2021
Dec 2 2021
Ping
Simplified diagnostic condition in TemplateDeclInstantiator::InstantiateTypedefNameDecl().