erichkeane added a comment.

@craig.topper  is the only one I know who can keep all of this straight, so 
perhaps he's willing to give a more in depth review?



================
Comment at: clang/include/clang/Basic/TargetInfo.h:207
   bool HasLongDouble;
+  bool Has80BitLongDouble;
   bool HasFPReturn;
----------------
This seems so shockingly x86 specific, I'm a little concerned we are doing it 
here.  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115337

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

Reply via email to