Skip to content

Commit 71f82bb

Browse files
[SPIRV] Use heterogenous lookups with std::map (NFC) (#115425)
Heterogenous lookups allow us to call find with StringRef, avoiding a temporary heap allocation of std::string.
1 parent 51e8f82 commit 71f82bb

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

llvm/lib/Target/SPIRV/SPIRVCommandLine.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020

2121
using namespace llvm;
2222

23-
static const std::map<std::string, SPIRV::Extension::Extension>
23+
static const std::map<std::string, SPIRV::Extension::Extension, std::less<>>
2424
SPIRVExtensionMap = {
2525
{"SPV_EXT_shader_atomic_float_add",
2626
SPIRV::Extension::Extension::SPV_EXT_shader_atomic_float_add},
@@ -98,7 +98,7 @@ bool SPIRVExtensionsParser::parse(cl::Option &O, llvm::StringRef ArgName,
9898
return O.error("Invalid extension list format: " + Token.str());
9999

100100
llvm::StringRef ExtensionName = Token.substr(1);
101-
auto NameValuePair = SPIRVExtensionMap.find(ExtensionName.str());
101+
auto NameValuePair = SPIRVExtensionMap.find(ExtensionName);
102102

103103
if (NameValuePair == SPIRVExtensionMap.end())
104104
return O.error("Unknown SPIR-V extension: " + Token.str());

0 commit comments

Comments
 (0)