[PATCH] D59257: [ARM] Run ARMParallelDSP in the IRPasses phase

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 13 10:00:39 PDT 2019


SjoerdMeijer added inline comments.


================
Comment at: lib/Target/ARM/ARMTargetMachine.cpp:411
+    addPass(createTypeBasedAAWrapperPass());
+    addPass(createScopedNoAliasAAWrapperPass());
+    addPass(createBasicAAWrapperPass());
----------------
Under CodeGenOpt::Aggressive, I don't see adding a cleanup pass like CSE being a problem (e.g. for compile-times).
However, I was wondering if we need all of these passes, like ScopedNoAliasAA.  I haven't looked at this pass before, so don't know what it is, do we need it? :-)


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

https://reviews.llvm.org/D59257





More information about the llvm-commits mailing list