[llvm] e617cf9 - [NFC] Restore original SmallString size for X86TargetMachine::getSubtargetImpl lookup

via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 22 11:19:54 PDT 2021


Author: serge-sans-paille
Date: 2021-03-22T19:19:46+01:00
New Revision: e617cf9576c9ffa41bbc35c4bd1c5651549d2c14

URL: https://github.com/llvm/llvm-project/commit/e617cf9576c9ffa41bbc35c4bd1c5651549d2c14
DIFF: https://github.com/llvm/llvm-project/commit/e617cf9576c9ffa41bbc35c4bd1c5651549d2c14.diff

LOG: [NFC] Restore original SmallString size for X86TargetMachine::getSubtargetImpl lookup

Better safe than sorry here, quoting Craig Topper:

> Clang passes a pretty lengthy feature string.

Added: 
    

Modified: 
    llvm/lib/Target/X86/X86TargetMachine.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/X86/X86TargetMachine.cpp b/llvm/lib/Target/X86/X86TargetMachine.cpp
index 9deae9d0e3061..32b90e31bec37 100644
--- a/llvm/lib/Target/X86/X86TargetMachine.cpp
+++ b/llvm/lib/Target/X86/X86TargetMachine.cpp
@@ -248,7 +248,7 @@ X86TargetMachine::getSubtargetImpl(const Function &F) const {
   StringRef FS =
       FSAttr.isValid() ? FSAttr.getValueAsString() : (StringRef)TargetFS;
 
-  SmallString<64> Key;
+  SmallString<512> Key;
   // The additions here are ordered so that the definitely short strings are
   // added first so we won't exceed the small size. We append the
   // much longer FS string at the end so that we only heap allocate at most


        


More information about the llvm-commits mailing list