[PATCH] Add sparc_le architecture (little-endian 32-bit Sparc).
    James Y Knight 
    jyknight at google.com
       
    Tue Mar 31 12:05:44 PDT 2015
    
    
  
In the description, you meant to say copied from Sparc V8, right? (as this has created a sparcv8 variant, not a v9 variant).
(and of course some test cases)
================
Comment at: lib/Target/Sparc/MCTargetDesc/SparcMCTargetDesc.cpp:54
@@ -53,1 +53,3 @@
 
+static MCAsmInfo *createSparc_leMCAsmInfo(const MCRegisterInfo &MRI,
+                                       StringRef TT) {
----------------
Why a new copy of this? Shouldn't it just use createSparcMCAsmInfo?
================
Comment at: lib/Target/Sparc/SparcTargetMachine.cpp:29
@@ +28,3 @@
+static std::string computeDataLayout(const Triple &T,
+				     bool is64Bit, char endianness) {
+  // Sparc is typically big endian, but some are little.
----------------
Perhaps less confusing to just have a bool isLittleEndian instead of a char passed in here?
http://reviews.llvm.org/D8741
EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
    
    
More information about the llvm-commits
mailing list