diff options
| author | Simon Pilgrim <llvm-dev@redking.me.uk> | 2019-10-16 17:21:50 +0000 |
|---|---|---|
| committer | Simon Pilgrim <llvm-dev@redking.me.uk> | 2019-10-16 17:21:50 +0000 |
| commit | e2163f96abb5905938115f8935702b1defe4f015 (patch) | |
| tree | 1d2d7eb5d75a3999cbe91934979102c6492bcf19 /llvm/lib/CodeGen/GlobalISel | |
| parent | f095b8c425ecf832d0a5ccdbaa02c27153f80a0c (diff) | |
| download | bcm5719-llvm-e2163f96abb5905938115f8935702b1defe4f015.tar.gz bcm5719-llvm-e2163f96abb5905938115f8935702b1defe4f015.zip | |
CombinerHelper - silence dead assignment warnings. NFCI.
Copy the NewAlignment value to Alignment first and then use that to update the stack frame object alignments.
llvm-svn: 375019
Diffstat (limited to 'llvm/lib/CodeGen/GlobalISel')
| -rw-r--r-- | llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp b/llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp index 974b500d4ab..e9de6c2c98d 100644 --- a/llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp +++ b/llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp @@ -779,11 +779,11 @@ bool CombinerHelper::optimizeMemset(MachineInstr &MI, Register Dst, Register Val Type *IRTy = getTypeForLLT(MemOps[0], C); unsigned NewAlign = (unsigned)DL.getABITypeAlignment(IRTy); if (NewAlign > Align) { + Align = NewAlign; unsigned FI = FIDef->getOperand(1).getIndex(); // Give the stack frame object a larger alignment if needed. - if (MFI.getObjectAlignment(FI) < NewAlign) - MFI.setObjectAlignment(FI, NewAlign); - Align = NewAlign; + if (MFI.getObjectAlignment(FI) < Align) + MFI.setObjectAlignment(FI, Align); } } @@ -907,11 +907,11 @@ bool CombinerHelper::optimizeMemcpy(MachineInstr &MI, Register Dst, NewAlign /= 2; if (NewAlign > Alignment) { + Alignment = NewAlign; unsigned FI = FIDef->getOperand(1).getIndex(); // Give the stack frame object a larger alignment if needed. - if (MFI.getObjectAlignment(FI) < NewAlign) - MFI.setObjectAlignment(FI, NewAlign); - Alignment = NewAlign; + if (MFI.getObjectAlignment(FI) < Alignment) + MFI.setObjectAlignment(FI, Alignment); } } @@ -1014,11 +1014,11 @@ bool CombinerHelper::optimizeMemmove(MachineInstr &MI, Register Dst, NewAlign /= 2; if (NewAlign > Alignment) { + Alignment = NewAlign; unsigned FI = FIDef->getOperand(1).getIndex(); // Give the stack frame object a larger alignment if needed. - if (MFI.getObjectAlignment(FI) < NewAlign) - MFI.setObjectAlignment(FI, NewAlign); - Alignment = NewAlign; + if (MFI.getObjectAlignment(FI) < Alignment) + MFI.setObjectAlignment(FI, Alignment); } } |
