eugenis accepted this revision.
eugenis added a comment.
This revision is now accepted and ready to land.

LGTM, please add a TODO for more passes



================
Comment at: clang/lib/CodeGen/BackendUtil.cpp:1291
+              // general purpose optimization passes.
+              FPM.addPass(EarlyCSEPass());
+            }
----------------
I did not know that MSan does not add the extra optimization passes in the new 
pass manager (see addGeneralOptsForMemorySanitizer). If I remember correctly, 
it used to give ~10% or more speedup on spec2006.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96406

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

Reply via email to