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

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

Reply via email to