[llvm] c3b03f3 - [AMDGPU] Drop const for value that is copied (NFC).

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 30 03:02:06 PDT 2020


Author: Florian Hahn
Date: 2020-03-30T10:59:59+01:00
New Revision: c3b03f3d0c3565a556c11841b7cee57418c48c1a

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

LOG: [AMDGPU] Drop const for value that is copied (NFC).

This fixes

    warning: loop variable 'Def' of type 'const llvm::Register' creates a copy from type 'const llvm::Register' [-Wrange-loop-analysis]

llvm::Register just contains a single unsigned and should be copied.

Reviewers: rampitec

Reviewed By: rampitec

Differential Revision: https://reviews.llvm.org/D77011

Added: 
    

Modified: 
    llvm/lib/Target/AMDGPU/SIPostRABundler.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/AMDGPU/SIPostRABundler.cpp b/llvm/lib/Target/AMDGPU/SIPostRABundler.cpp
index acb0fbfe812d..3a64d519d105 100644
--- a/llvm/lib/Target/AMDGPU/SIPostRABundler.cpp
+++ b/llvm/lib/Target/AMDGPU/SIPostRABundler.cpp
@@ -76,7 +76,7 @@ bool SIPostRABundler::isDependentLoad(const MachineInstr &MI) const {
     if (!Op.isReg())
       continue;
     Register Reg = Op.getReg();
-    for (const Register Def : Defs)
+    for (Register Def : Defs)
       if (TRI->regsOverlap(Reg, Def))
         return true;
   }


        


More information about the llvm-commits mailing list