[PATCH] D46396: Optionally simplify basic blocks introduced by AtomicExpandPass

Ahmed Bougacha via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 4 11:42:39 PDT 2018


ab added a reviewer: t.p.northover.
ab added inline comments.


================
Comment at: lib/Target/AArch64/AArch64TargetMachine.cpp:376
   if (TM->getOptLevel() != CodeGenOpt::None && EnableAtomicTidy)
     addPass(createCFGSimplificationPass(1, true, true, false, true));
 
----------------
This already does the simplification, no?  How about adding the pass for Hexagon as well?  I suppose this patch limits it to a smaller set of blocks, but on the other hand it's nice not to have to deal with it in the pass.


Repository:
  rL LLVM

https://reviews.llvm.org/D46396





More information about the llvm-commits mailing list