mibintc marked 2 inline comments as done.
mibintc added inline comments.

================
Comment at: clang/lib/Parse/ParsePragma.cpp:111
       PP.Diag(Tok, diag::warn_stdc_fenv_access_not_supported);
+      return;
     }
----------------
Modified this to return instead of making changes which will ultimately modify 
the floating point state


================
Comment at: clang/test/CodeGen/fp-floatcontrol-pragma.cpp:61
+  z = z * z;
+//CHECK: = fmul float
+}
----------------
I used a modified version of @uabelho 's test. I verified that before this bug 
fix a constrained intrinsic was being created for this multiply.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79510



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to