steven.zhang added inline comments.

================
Comment at: clang/lib/Basic/Targets/PPC.h:86
+
+    HasStrictFP = true;
   }
----------------
nemanjai wrote:
> I don't think we need this for now. Close is not quite there. @steven.zhang I 
> would prefer that we initially turn this off and only flip it on once the 
> support is complete.
> Also, is the support that is currently under development for both 32 and 64 
> bit architectures? If it is 64 bit only, then we can enable it only there 
> once it is done.
Yes, it hasn't been finished yet. Agree that we can turn on it later when it is 
done.


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

https://reviews.llvm.org/D80952



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

Reply via email to