[PATCH] D139008: Intrinsics: Make is_fpclass and arithmetic_fence speculatable

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/include/llvm/IR/Intrinsics.td:715
+  def int_arithmetic_fence : DefaultAttrsIntrinsic<[llvm_anyfloat_ty], [LLVMMatchType<0>],
+                                                   [IntrNoMem]>;
 
----------------
jdoerfert wrote:
> The subject line says this is now speculatable but the definition here is not. I'm confused.
> Given the description, this "fence" doesn't actually do anything except provide a semi-permeable value pass-through, right? Then I agree this should be speculatable.
The math instructions here are wrapped in a let IntrProperties block above. arithmetic_fence picks up willreturn also 


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

https://reviews.llvm.org/D139008