[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