aeubanks added a comment.

In D93656#2468834 <https://reviews.llvm.org/D93656#2468834>, @hoy wrote:

> In D93656#2468821 <https://reviews.llvm.org/D93656#2468821>, @aeubanks wrote:
>
>> Also it looks like this is doing 2 different things, the moving of things 
>> from Clang to LLVM's PassBuilder, and separately the change to the pass 
>> itself. Can these be separated?
>
> I'm not sure about a good way to separate them. There are Clang tests that 
> may fail with removing the pass from clang while not adding it 
> correspondingly in llvm. Adding the pass in llvm while not removing it from 
> Clang may cause the pass to run twice which may also fail the Clang tests. 
> What do you think?

I mean keep that in one change, but separate out the change to 
llvm/lib/Transforms/Utils/UniqueInternalLinkageNames.cpp and 
DebugInfoMetadata.h.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93656/new/

https://reviews.llvm.org/D93656

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to