[PATCH] D112178: [LazyCallGraph] Skip blockaddresses

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 1 11:00:19 PDT 2021


nickdesaulniers accepted this revision.
nickdesaulniers added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/test/Transforms/Attributor/IPConstantProp/dangling-block-address.ll:37
 define internal void @bar(i32* nocapture %pc) nounwind readonly {
-; IS__CGSCC_OPM: Function Attrs: nounwind readonly
-; IS__CGSCC_OPM-LABEL: define {{[^@]+}}@bar
----------------
aeubanks wrote:
> aeubanks wrote:
> > nickdesaulniers wrote:
> > > Can we remove `IS__CGSCC_OPM` from the `RUN` lines, too? Looks like this change removes all references? Perhaps we can get rid of the OPM RUN lines? And pre-commit that?
> > will do
> apparently it's a bit more complicated than I originally thought: https://reviews.llvm.org/D112184
> 
> can we punt on this for now?
Can you remove one run line from one test?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112178



More information about the llvm-commits mailing list