[PATCH] D66132: [CodeGen] Add `isSpeculativeExecutionForbidden()` to `TargetInstrInfo` (PRR42405)

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/TargetInstrInfo.h:1026-1028
+  /// This prevents MIR code from SimplePRE optimization leading to possible
+  /// speculative execution of instructions.
+  virtual bool isSpeculativeExecutionForbidden() const { return false; }
----------------
Regardless of my bikeshedding if this is the correct fix or not,
this should be renamed to be less misleading.
Just name it `shouldDoPartialRedundancyElimination()` ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66132