[PATCH] D124892: [DWARF] Fix split dwarf debug inlining in mix mode situation.
Alexander Yermolovich via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 5 16:01:05 PDT 2022
ayermolo added a comment.
In D124892#3494854 <https://reviews.llvm.org/D124892#3494854>, @dblaikie wrote:
> Yeah, mutating the `DICompileUnit` at this point seems a bad idea - it could end up inconsistent between things that happened before the mutation V after, etc.
>
> Also the testing isn't suitable - LLVM tests can't depend on clang and the two IR files could be linked & the single IR result could be the test case? (it should just run llvm-mc - check other debug info tests). With the simplified/linked test, I can take a look and see if I can understand what's broken/how it might be suitable to fix it.
Ah thanks!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D124892/new/
https://reviews.llvm.org/D124892
More information about the llvm-commits
mailing list