diff options
author | Bill Wendling <isanbard@gmail.com> | 2013-01-02 23:45:09 +0000 |
---|---|---|
committer | Bill Wendling <isanbard@gmail.com> | 2013-01-02 23:45:09 +0000 |
commit | 278bac3fba5ae8ae620c961621493e6e2ad6f953 (patch) | |
tree | ad78b1e127ef0f0d1f8dc6274df89432059bee9e /include | |
parent | d40758b24ebab5777131533d9369e707fc852594 (diff) | |
download | llvm-278bac3fba5ae8ae620c961621493e6e2ad6f953.tar.gz llvm-278bac3fba5ae8ae620c961621493e6e2ad6f953.tar.bz2 llvm-278bac3fba5ae8ae620c961621493e6e2ad6f953.tar.xz |
An intermediate step in the Attributes rewrite.
Modify the AttrBuilder class to store the attributes as a set instead of as a
bit mask. The Attribute class will represent only one attribute instead of a
collection of attributes.
This is the wave of the future!
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@171427 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'include')
-rw-r--r-- | include/llvm/IR/Attributes.h | 68 |
1 files changed, 46 insertions, 22 deletions
diff --git a/include/llvm/IR/Attributes.h b/include/llvm/IR/Attributes.h index c28f0bd090..19df4eb33a 100644 --- a/include/llvm/IR/Attributes.h +++ b/include/llvm/IR/Attributes.h @@ -17,8 +17,7 @@ #define LLVM_ATTRIBUTES_H #include "llvm/ADT/ArrayRef.h" -#include "llvm/Support/MathExtras.h" -#include <cassert> +#include "llvm/ADT/DenseSet.h" #include <string> namespace llvm { @@ -154,63 +153,88 @@ public: //===----------------------------------------------------------------------===// /// \class +/// \brief Provide DenseMapInfo for Attribute::AttrKind. +template<> struct DenseMapInfo<Attribute::AttrKind> { + static inline Attribute::AttrKind getEmptyKey() { + return Attribute::AttrKind(-1); + } + static inline Attribute::AttrKind getTombstoneKey() { + return Attribute::AttrKind(~0UL - 1L); + } + static unsigned getHashValue(const Attribute::AttrKind &Val) { + return (unsigned)(Val * 37UL); + } + static bool isEqual(const Attribute::AttrKind &LHS, + const Attribute::AttrKind &RHS) { + return LHS == RHS; + } +}; + +//===----------------------------------------------------------------------===// +/// \class /// \brief This class is used in conjunction with the Attribute::get method to /// create an Attribute object. The object itself is uniquified. The Builder's /// value, however, is not. So this can be used as a quick way to test for /// equality, presence of attributes, etc. class AttrBuilder { - uint64_t Bits; + DenseSet<Attribute::AttrKind> AttrSet; + uint64_t Alignment; + uint64_t StackAlignment; + + uint64_t Bits; // FIXME: Remove after encoding the attr list in the bc file. public: - AttrBuilder() : Bits(0) {} + AttrBuilder() : Alignment(0), StackAlignment(0), Bits(0) {} explicit AttrBuilder(uint64_t B) : Bits(B) {} AttrBuilder(const Attribute &A) : Bits(A.getBitMask()) {} - void clear() { Bits = 0; } + /// \brief Clear out the builder's internals. + void clear(); - /// addAttribute - Add an attribute to the builder. + /// \brief Add an attribute to the builder. AttrBuilder &addAttribute(Attribute::AttrKind Val); - /// removeAttribute - Remove an attribute from the builder. + /// \brief Remove an attribute from the builder. AttrBuilder &removeAttribute(Attribute::AttrKind Val); - /// addAttribute - Add the attributes from A to the builder. + /// \brief Add the attributes from A to the builder. AttrBuilder &addAttributes(const Attribute &A); - /// removeAttribute - Remove the attributes from A from the builder. + /// \brief Remove the attributes from A from the builder. AttrBuilder &removeAttributes(const Attribute &A); /// \brief Return true if the builder has the specified attribute. bool contains(Attribute::AttrKind A) const; - /// hasAttributes - Return true if the builder has IR-level attributes. + /// \brief Return true if the builder has IR-level attributes. bool hasAttributes() const; - /// hasAttributes - Return true if the builder has any attribute that's in the + /// \brief Return true if the builder has any attribute that's in the /// specified attribute. bool hasAttributes(const Attribute &A) const; - /// hasAlignmentAttr - Return true if the builder has an alignment attribute. + /// \brief Return true if the builder has an alignment attribute. bool hasAlignmentAttr() const; - /// getAlignment - Retrieve the alignment attribute, if it exists. + /// \brief Retrieve the alignment attribute, if it exists. uint64_t getAlignment() const; - /// getStackAlignment - Retrieve the stack alignment attribute, if it exists. + /// \brief Retrieve the stack alignment attribute, if it exists. uint64_t getStackAlignment() const; - /// addAlignmentAttr - This turns an int alignment (which must be a power of - /// 2) into the form used internally in Attribute. + /// \brief This turns an int alignment (which must be a power of 2) into the + /// form used internally in Attribute. AttrBuilder &addAlignmentAttr(unsigned Align); - /// addStackAlignmentAttr - This turns an int stack alignment (which must be a - /// power of 2) into the form used internally in Attribute. + /// \brief This turns an int stack alignment (which must be a power of 2) into + /// the form used internally in Attribute. AttrBuilder &addStackAlignmentAttr(unsigned Align); - /// addRawValue - Add the raw value to the internal representation. + /// \brief Add the raw value to the internal representation. + /// /// N.B. This should be used ONLY for decoding LLVM bitcode! AttrBuilder &addRawValue(uint64_t Val); - /// @brief Remove attributes that are used on functions only. + /// \brief Remove attributes that are used on functions only. void removeFunctionOnlyAttrs() { removeAttribute(Attribute::NoReturn) .removeAttribute(Attribute::NoUnwind) @@ -236,10 +260,10 @@ public: uint64_t getBitMask() const { return Bits; } - bool operator==(const AttrBuilder &B) { + bool operator==(const AttrBuilder &B) const { return Bits == B.Bits; } - bool operator!=(const AttrBuilder &B) { + bool operator!=(const AttrBuilder &B) const { return Bits != B.Bits; } }; |