kristina added a comment.

Let me know if you need this committed.



================
Comment at: lib/Basic/Targets/OSTargets.h:260
     DefineStd(Builder, "unix", Opts);
+    if (this->HasFloat128) {
+      Builder.defineMacro("__FLOAT128__");
----------------
One tiny style-related nitpick, no need for braces here, can just remove them 
after applying the patch.


Repository:
  rC Clang

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

https://reviews.llvm.org/D54901



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

Reply via email to