[PATCH] D115620: [AArch64] Lowering and legalization of strict FP16

[Dave Green via Phabricator via cfe-commits]

dmgreen added a comment.

I might have misunderstood which this patch was. Can you move the clang test into D118259 <https://reviews.llvm.org/D118259> with the other? It seems like the same problem, and it looks like there should be enough llc tests to cover all the cases.



================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:1386
+
+  IsStrictFPEnabled = true;
 }
----------------
What are the ramifications of setting this to true?


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

https://reviews.llvm.org/D115620