[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