[PATCH] Fix ELF st_other flag.

Rafael Ávila de Espíndola rafael.espindola at gmail.com
Wed Dec 4 09:03:10 PST 2013


  LGTM


================
Comment at: include/llvm/MC/MCELFSymbolFlags.h:28
@@ -28,1 +27,3 @@
+    ELF_STO_Shift   = 10, // Shift value for STO_* flags.
+    ELF_Other_Shift = 16  // Shift value for other flags.
   };
----------------
Maybe say "Shift value for llvm local flags, not part of the final object file".


http://llvm-reviews.chandlerc.com/D2329

BRANCH
  fix-st-other

ARCANIST PROJECT
  llvm



More information about the llvm-commits mailing list