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

Kevin P. Neal via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 24 11:41:43 PDT 2019


kpn added inline comments.
Herald added a subscriber: wuzish.


================
Comment at: lib/CodeGen/SelectionDAG/LegalizeDAG.cpp:2790
+        == TargetLowering::Legal)
+      break;
     Tmp1 = EmitStackConvert(Node->getOperand(1), 
----------------
In what way does it not honor the strict properties?

Also, wouldn't it be a bug if we were asked to expand a strict node when the non-strict is legal?


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