hfinkel added inline comments.

================
Comment at: lib/Basic/TargetInfo.cpp:277
+    LongDoubleWidth = 64;
+    LongDoubleAlign = 32;
+    LongDoubleFormat = &llvm::APFloat::IEEEdouble;
----------------
This seems wrong for targets in general. Maybe this should be, instead of 32 
always, 64 on 64-bit targets?


https://reviews.llvm.org/D6260



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D6260: Add -mlong... Chandler Carruth via Phabricator via cfe-commits
    • [PATCH] D6260: Add -... Hal Finkel via Phabricator via cfe-commits

Reply via email to