dblaikie added a comment. In D94019#2478047 <https://reviews.llvm.org/D94019#2478047>, @hoy wrote:
> In D94019#2478045 <https://reviews.llvm.org/D94019#2478045>, @dblaikie wrote: > >> Please add a clang test for this. > > There is the original clang test `unique-internal-linkage-names.cpp` that > still works with the change here. What kind of new test would you like? Something that tests this change (something that would fail before this patch is applied, and passes afterwards - demonstrating the change in behavior). Either something like the LLVM test, testing the pass sequence, or something with very simple IR (something that can robustly demonstrate the change in optimization behavior due to this patch and will be resilient to as many other changes to LLVM as possible (ie: something that captures the essence of this change)). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94019/new/ https://reviews.llvm.org/D94019 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
