krisb added a comment. Tested the whole patchset with various combinations of
- opt levels (O0, O2 <https://reviews.llvm.org/owners/package/2/>, O3 <https://reviews.llvm.org/owners/package/3/>, ThinLTO, FullLTO), - debug info options (-fdebug-types-section, -gline-tables-only, -fno-eliminate-unused-debug-types -gsplit-dwarf, -fsplit-dwarf-inlining), - and some additional checks / asserts (like additional verification of abstract / concrete out-of-line tree to be sure local entities are not duplicated, do not go to the wrong tree) on clang bootstrapping build. No issues so far (except PR55680, which caused not by this and related patches). This still doesn't guarantee that the patch accounts all the corner cases and no issues would be exposed later, but seems good to start with. Removing [WIP] status. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125693/new/ https://reviews.llvm.org/D125693 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
