maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[Hexagon] Fix cppcheck reduce variable scope warnings. NFCI. 

Also fixes a static analyzer "Value stored to 'S2' during its initialization is never read" warning.

llvm-svn: 360244
This commit is contained in:
Simon Pilgrim 2019-05-08 11:02:46 +00:00
parent d52a542e4c
commit ec58090491
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
llvm/lib/Target/Hexagon/HexagonStoreWidening.cpp
View File

@ -337,8 +337,7 @@ bool HexagonStoreWidening::selectStores(InstrGroup::iterator Begin,
return false;
OG.push_back(FirstMI);
MachineInstr *S1 = FirstMI, *S2 = *(Begin+1);
InstrGroup::iterator I = Begin+1;
MachineInstr *S1 = FirstMI;
// Pow2Num will be the largest number of elements in OG such that the sum
// of sizes of stores 0...Pow2Num-1 will be a power of 2.
@ -350,8 +349,8 @@ bool HexagonStoreWidening::selectStores(InstrGroup::iterator Begin,
// does not exceed the limit (MaxSize).
// Keep track of when the total size covered is a power of 2, since
// this is a size a single store can cover.
while (I != End) {
S2 = *I;
for (InstrGroup::iterator I = Begin + 1; I != End; ++I) {
MachineInstr *S2 = *I;
// Stores are sorted, so if S1 and S2 are not adjacent, there won't be
// any other store to fill the "hole".
if (!storesAreAdjacent(S1, S2))
@ -371,7 +370,6 @@ bool HexagonStoreWidening::selectStores(InstrGroup::iterator Begin,
break;
S1 = S2;
++I;
}
// The stores don't add up to anything that can be widened. Clean up.