craig.topper added inline comments.

================
Comment at: llvm/test/CodeGen/X86/O3-pipeline.ll:147
+; CHECK-NEXT:       X86 Load Value Injection (LVI) Load Hardening
 ; CHECK-NEXT:       Fixup Statepoint Caller Saved
 ; CHECK-NEXT:       PostRA Machine Sink
----------------
I'm curious what happens if we add AU.setPreservesCFG() to getAnalysisUsage in 
FixupStatepointCallerSaved.cpp  From a quick look through that pass it doesn't 
look like it changes the Machine CFG.

PostRA Machine Sink already preserves CFG. So I think that should remove the 
dominator tree construction after PostRA machine sink.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75936



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

Reply via email to