[PATCH] D121292: [VP] Add vp.fcmp comparison intrinsic and docs

Mon Mar 21 05:00:03 PDT 2022

frasercrmck added a comment.

In D121292#3385541 <https://reviews.llvm.org/D121292#3385541>, @simoll wrote:

> What we really need is something that tells us whether the replacement for the VP intrinsic is speculatable (not the VP intrinsic itself).
> Something like `isSafeToSpeculativelyExecute` with just the opcode and the operands.

Yeah, that makes sense. That needs to be a separate patch, however, so at the very least I should not be adding "speculatable" to these new intrinsics. This only really comes up as a problem in D121594 <https://reviews.llvm.org/D121594>.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121292