[llvm] ecea837 - [TargetLibraryInfo] Use std::move (NFC) (#95671)

via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 15 14:02:45 PDT 2024


Author: Kazu Hirata
Date: 2024-06-15T14:02:42-07:00
New Revision: ecea8371ff03c15fb3dc27ee4108b98335fd2d63

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

LOG: [TargetLibraryInfo] Use std::move (NFC) (#95671)

The std::move here saves 0.11% of heap allocations during the
compilation of a large preprocessed file, namely X86ISelLowering.cpp,
for the X86 target.

Added: 
    

Modified: 
    llvm/include/llvm/Analysis/TargetLibraryInfo.h

Removed: 
    


################################################################################
diff  --git a/llvm/include/llvm/Analysis/TargetLibraryInfo.h b/llvm/include/llvm/Analysis/TargetLibraryInfo.h
index f5da222d11f55..ee19bc816b4e0 100644
--- a/llvm/include/llvm/Analysis/TargetLibraryInfo.h
+++ b/llvm/include/llvm/Analysis/TargetLibraryInfo.h
@@ -316,11 +316,12 @@ class TargetLibraryInfo {
   // Provide value semantics.
   TargetLibraryInfo(const TargetLibraryInfo &TLI) = default;
   TargetLibraryInfo(TargetLibraryInfo &&TLI)
-      : Impl(TLI.Impl), OverrideAsUnavailable(TLI.OverrideAsUnavailable) {}
+      : Impl(TLI.Impl),
+        OverrideAsUnavailable(std::move(TLI.OverrideAsUnavailable)) {}
   TargetLibraryInfo &operator=(const TargetLibraryInfo &TLI) = default;
   TargetLibraryInfo &operator=(TargetLibraryInfo &&TLI) {
     Impl = TLI.Impl;
-    OverrideAsUnavailable = TLI.OverrideAsUnavailable;
+    OverrideAsUnavailable = std::move(TLI.OverrideAsUnavailable);
     return *this;
   }
 


        


More information about the llvm-commits mailing list