[PATCH] D65226: [Strict FP] Allow custom operation actions

Cameron McInally via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 25 10:16:36 PDT 2019


cameron.mcinally added a comment.

I don't see a problem with scalarizing strict ops for Custom lowered nodes that would otherwise be legal. It's not ideal, I suppose, but if a target doesn't support the strict nodes in the backend, then it probably shouldn't be using the experimental intrinsics anyway.

I don't really have a strong opinion on this though, so will leave it to others that may have one...


Repository:
  rL LLVM

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

https://reviews.llvm.org/D65226





More information about the llvm-commits mailing list