[llvm] r212834 - [MC] Constify MCELF::GetVisibility and MCELF::getOther

Ulrich Weigand ulrich.weigand at de.ibm.com
Fri Jul 11 10:34:45 PDT 2014


Author: uweigand
Date: Fri Jul 11 12:34:44 2014
New Revision: 212834

URL: http://llvm.org/viewvc/llvm-project?rev=212834&view=rev
Log:
[MC] Constify MCELF::GetVisibility and MCELF::getOther

These two routines didn't take a "const MCSymbolData &SD"
like the other MCELF::Get routines for some reason ...


Modified:
    llvm/trunk/include/llvm/MC/MCELF.h
    llvm/trunk/lib/MC/MCELF.cpp

Modified: llvm/trunk/include/llvm/MC/MCELF.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/MC/MCELF.h?rev=212834&r1=212833&r2=212834&view=diff
==============================================================================
--- llvm/trunk/include/llvm/MC/MCELF.h (original)
+++ llvm/trunk/include/llvm/MC/MCELF.h Fri Jul 11 12:34:44 2014
@@ -27,9 +27,9 @@ class MCELF {
   static void SetType(MCSymbolData &SD, unsigned Type);
   static unsigned GetType(const MCSymbolData &SD);
   static void SetVisibility(MCSymbolData &SD, unsigned Visibility);
-  static unsigned GetVisibility(MCSymbolData &SD);
+  static unsigned GetVisibility(const MCSymbolData &SD);
   static void setOther(MCSymbolData &SD, unsigned Other);
-  static unsigned getOther(MCSymbolData &SD);
+  static unsigned getOther(const MCSymbolData &SD);
 };
 
 }

Modified: llvm/trunk/lib/MC/MCELF.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/MC/MCELF.cpp?rev=212834&r1=212833&r2=212834&view=diff
==============================================================================
--- llvm/trunk/lib/MC/MCELF.cpp (original)
+++ llvm/trunk/lib/MC/MCELF.cpp Fri Jul 11 12:34:44 2014
@@ -61,7 +61,7 @@ void MCELF::SetVisibility(MCSymbolData &
   SD.setFlags(OtherFlags | (Visibility << ELF_STV_Shift));
 }
 
-unsigned MCELF::GetVisibility(MCSymbolData &SD) {
+unsigned MCELF::GetVisibility(const MCSymbolData &SD) {
   unsigned Visibility =
     (SD.getFlags() & (0x3 << ELF_STV_Shift)) >> ELF_STV_Shift;
   assert(Visibility == ELF::STV_DEFAULT || Visibility == ELF::STV_INTERNAL ||
@@ -76,7 +76,7 @@ void MCELF::setOther(MCSymbolData &SD, u
   SD.setFlags(OtherFlags | (Other << ELF_STO_Shift));
 }
 
-unsigned MCELF::getOther(MCSymbolData &SD) {
+unsigned MCELF::getOther(const MCSymbolData &SD) {
   unsigned Other =
     (SD.getFlags() & (0x3f << ELF_STO_Shift)) >> ELF_STO_Shift;
   return Other;





More information about the llvm-commits mailing list