[PATCH] D115337: [X86][clang] Put the update of HasLongDouble into TargetInfo::adjust

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 8 06:37:47 PST 2021


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



More information about the cfe-commits mailing list