[llvm] b2e78a0 - [NFC] Use SmallString instead of std::string for the AttrBuilder
via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 16 05:34:49 PDT 2021
Author: serge-sans-paille
Date: 2021-03-16T13:34:14+01:00
New Revision: b2e78a061c06546c42a977071047cd9da2194a32
URL: https://github.com/llvm/llvm-project/commit/b2e78a061c06546c42a977071047cd9da2194a32
DIFF: https://github.com/llvm/llvm-project/commit/b2e78a061c06546c42a977071047cd9da2194a32.diff
LOG: [NFC] Use SmallString instead of std::string for the AttrBuilder
This avoids a few unnecessary conversion from StringRef to std::string, and a
bunch of extra allocation thanks to the SmallString.
Differential Revision: https://reviews.llvm.org/D98190
Added:
Modified:
llvm/include/llvm/IR/Attributes.h
llvm/lib/IR/Attributes.cpp
Removed:
################################################################################
diff --git a/llvm/include/llvm/IR/Attributes.h b/llvm/include/llvm/IR/Attributes.h
index b4056540663f..20f5cf1b1917 100644
--- a/llvm/include/llvm/IR/Attributes.h
+++ b/llvm/include/llvm/IR/Attributes.h
@@ -18,6 +18,7 @@
#include "llvm-c/Types.h"
#include "llvm/ADT/ArrayRef.h"
#include "llvm/ADT/Optional.h"
+#include "llvm/ADT/SmallString.h"
#include "llvm/ADT/StringRef.h"
#include "llvm/ADT/iterator_range.h"
#include "llvm/Config/llvm-config.h"
@@ -756,7 +757,7 @@ template <> struct DenseMapInfo<AttributeList> {
/// equality, presence of attributes, etc.
class AttrBuilder {
std::bitset<Attribute::EndAttrKinds> Attrs;
- std::map<std::string, std::string, std::less<>> TargetDepAttrs;
+ std::map<SmallString<32>, SmallString<32>, std::less<>> TargetDepAttrs;
MaybeAlign Alignment;
MaybeAlign StackAlignment;
uint64_t DerefBytes = 0;
@@ -921,7 +922,7 @@ class AttrBuilder {
bool empty() const { return Attrs.none(); }
// Iterators for target-dependent attributes.
- using td_type = std::pair<std::string, std::string>;
+ using td_type = decltype(TargetDepAttrs)::value_type;
using td_iterator = decltype(TargetDepAttrs)::iterator;
using td_const_iterator = decltype(TargetDepAttrs)::const_iterator;
using td_range = iterator_range<td_iterator>;
diff --git a/llvm/lib/IR/Attributes.cpp b/llvm/lib/IR/Attributes.cpp
index c98e5c36a518..18c2f3aad5f0 100644
--- a/llvm/lib/IR/Attributes.cpp
+++ b/llvm/lib/IR/Attributes.cpp
@@ -1624,7 +1624,7 @@ AttrBuilder &AttrBuilder::addAttribute(Attribute Attr) {
}
AttrBuilder &AttrBuilder::addAttribute(StringRef A, StringRef V) {
- TargetDepAttrs[std::string(A)] = std::string(V);
+ TargetDepAttrs[A] = V;
return *this;
}
More information about the llvm-commits
mailing list