From 1a8319cbef7d96a40ef3728ddcd2d5ad9668a2aa Mon Sep 17 00:00:00 2001 From: Pete Cooper Date: Mon, 8 Jun 2015 17:17:30 +0000 Subject: [PATCH] Make flags get/set method protected so that all flags have to be managed by subclasses. MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit All of ELF, COFF and MachO now manipulate the flags in helpers so we don't need anyone to read the flags directly, but instead via those helpers. Reviewed by Rafael Espíndola. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@239317 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/llvm/MC/MCSymbol.h | 23 ++++++++++++----------- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/include/llvm/MC/MCSymbol.h b/include/llvm/MC/MCSymbol.h index 5710051840e..71d29c6e442 100644 --- a/include/llvm/MC/MCSymbol.h +++ b/include/llvm/MC/MCSymbol.h @@ -290,17 +290,6 @@ public: /// Is this a 'common' symbol. bool isCommon() const { return CommonAlign != -1U; } - /// Get the (implementation defined) symbol flags. - uint32_t getFlags() const { return Flags; } - - /// Set the (implementation defined) symbol flags. - void setFlags(uint32_t Value) const { Flags = Value; } - - /// Modify the flags via a mask - void modifyFlags(uint32_t Value, uint32_t Mask) const { - Flags = (Flags & ~Mask) | Value; - } - MCFragment *getFragment() const { if (!HasFragment) return nullptr; @@ -322,6 +311,18 @@ public: /// dump - Print the value to stderr. void dump() const; + +protected: + /// Get the (implementation defined) symbol flags. + uint32_t getFlags() const { return Flags; } + + /// Set the (implementation defined) symbol flags. + void setFlags(uint32_t Value) const { Flags = Value; } + + /// Modify the flags via a mask + void modifyFlags(uint32_t Value, uint32_t Mask) const { + Flags = (Flags & ~Mask) | Value; + } }; inline raw_ostream &operator<<(raw_ostream &OS, const MCSymbol &Sym) { -- 2.34.1