[PATCH] D27662: [triple+llc+llvm-mc] Make triple and ABI name consistent

Simon Dardis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 18 12:38:02 PST 2017


sdardis added a comment.

This looks ok to me as a starting point to begin rely only on the triple for n32/n64. @echristo ?



================
Comment at: include/llvm/ADT/Triple.h:201-207
+    AndroidABI32,
+    AndroidABI64,
+    ABI32,
+    ABIN32,
+    ABI64,
+    GNUABI32,
+    GNUABIN32,
----------------
I think for the moment, AndroidABI32, ABI*, and GNUABI32 are superfluous. We can infer enough and/or rely on clang to disambiguate the precise target.  


Repository:
  rL LLVM

https://reviews.llvm.org/D27662





More information about the llvm-commits mailing list