Fixes compilation problem on GCC 14

bokrzesi · pszymich · commit a8f626d1b851 · 2024-08-21T09:28:13.000+02:00
GCC detected dangerous pattern which required refactor in order to compile. (cherry picked from commit 03a6d27)
diff --git a/IGC/Compiler/Optimizer/MCSOptimization.cpp b/IGC/Compiler/Optimizer/MCSOptimization.cpp
@@ -182,6 +182,7 @@ void MCSOptimization::visitCallInst(llvm::CallInst& I)
             for (auto BB : useBlocks)
             {
                 std::vector<LdmsInstrinsic*> ldmsInstsToMove;
+                std::vector<LdmsInstrinsic*> ldmsInstsToClub;
                 for (auto inst = BB->begin(); inst != BB->end(); inst++)
                 {
                     if (LdmsInstrinsic * ldmsIntr = dyn_cast<LdmsInstrinsic>(inst))
@@ -203,7 +204,7 @@ void MCSOptimization::visitCallInst(llvm::CallInst& I)
 
                 while (!allInstsWillBeMoved)
                 {
-                    std::vector<LdmsInstrinsic*> ldmsInstsToClub;
+                    ldmsInstsToClub.clear();
                     //Threshold is more than # of insts that are to be moved. So move all.
                     if (instClubThreshold >= static_cast<int>(ldmsInstsToMove.size()))
                     {

Original file line number	Diff line number	Diff line change
`@@ -182,6 +182,7 @@ void MCSOptimization::visitCallInst(llvm::CallInst& I)`
`182`	`182`	`for (auto BB : useBlocks)`
`183`	`183`	`{`
`184`	`184`	`std::vector<LdmsInstrinsic*> ldmsInstsToMove;`
	`185`	`+ std::vector<LdmsInstrinsic*> ldmsInstsToClub;`
`185`	`186`	`for (auto inst = BB->begin(); inst != BB->end(); inst++)`
`186`	`187`	`{`
`187`	`188`	`if (LdmsInstrinsic * ldmsIntr = dyn_cast<LdmsInstrinsic>(inst))`
`@@ -203,7 +204,7 @@ void MCSOptimization::visitCallInst(llvm::CallInst& I)`
`203`	`204`
`204`	`205`	`while (!allInstsWillBeMoved)`
`205`	`206`	`{`
`206`		`- std::vector<LdmsInstrinsic*> ldmsInstsToClub;`
	`207`	`+ ldmsInstsToClub.clear();`
`207`	`208`	`//Threshold is more than # of insts that are to be moved. So move all.`
`208`	`209`	`if (instClubThreshold >= static_cast<int>(ldmsInstsToMove.size()))`
`209`	`210`	`{`