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

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added reviewers: t.p.northover, aymanmus, asbirlea, efriedma.
lebedev.ri added a comment.

After reading through https://bugs.llvm.org/show_bug.cgi?id=42405 i'm not confident that this is the correct solution.
It reads to me as-if this speculative execution in PRE being done with no proper legality checks.
In particular, i'd think you want to check a white-list in `MachineCSE::isPRECandidate()`,
much like `llvm::isSafeToSpeculativelyExecute()` is for middle-end.

Unless i'm really looking in the wrong place, there is no such thing here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66132