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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits