[PATCH] D75939: [x86][seses] Introduce SESES pass for LVI

Scott Constable via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 27 12:22:49 PDT 2020


sconstab added inline comments.


================
Comment at: llvm/lib/Target/X86/X86SpeculativeExecutionSideEffectSuppression.cpp:87
+    return false;
+
+  LLVM_DEBUG(dbgs() << "********** " << getPassName() << " : " << MF.getName()
----------------
There should probably be a CFI check here, e.g.:
```
if (!Subtarget.useLVIControlFlowIntegrity()) {
  report_fatal_error("SESES must be used with -mlvi-cfi", false);
}
```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75939





More information about the llvm-commits mailing list