[PATCH] D126479: [Clang] Allow 'Complex float __attribute__((mode(HC)))'

Tom Honermann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 31 11:01:07 PDT 2022


tahonermann added inline comments.


================
Comment at: clang/include/clang/Basic/TargetInfo.h:60
+  Ibm128,
+  Half
 };
----------------
The existing enumerators were ordered according to precision. Consider moving `Half` to before `Float` if doing so doesn't cause any problems (I would hope there is no dependence on the assigned enumerator values anywhere; if there is, then it would be helpful to add a comment about that here).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126479



More information about the cfe-commits mailing list