vitalybuka added a comment.

In D112732#3107387 <https://reviews.llvm.org/D112732#3107387>, @aeubanks wrote:

> can we rename "asan-module" to "asan" after this change? and remove the extra 
> "asan-pipeline"/"asan-function-pipeline" parsing callbacks in 
> NewPMDriver.cpp? and previously we had `-passes=require<asan-globals-md>` 
> because a function pass couldn't invoke a module analysis, but that's no 
> longer relevant, so cleaning those up from RUN lines would also be good

That was idea, but I changed my mind and recommended to do that in a separate 
patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112732

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

Reply via email to