diff options
| author | Diego Novillo <dnovillo@google.com> | 2015-11-11 16:39:22 +0000 |
|---|---|---|
| committer | Diego Novillo <dnovillo@google.com> | 2015-11-11 16:39:22 +0000 |
| commit | 0767ae5896cf05bcc8146fd81302920dba6ac30d (patch) | |
| tree | 4f4de11f0d895659188af1cd5c2ac2e5701d6b22 | |
| parent | 20a4c0c205d7c952cc58680bc5aa98f1424a24fd (diff) | |
| download | llvm-0767ae5896cf05bcc8146fd81302920dba6ac30d.zip llvm-0767ae5896cf05bcc8146fd81302920dba6ac30d.tar.gz llvm-0767ae5896cf05bcc8146fd81302920dba6ac30d.tar.bz2 | |
Properly fix unused variable in disable-assert builds.
I missed the side-effects of ParseBFI in my previous attempt (r252748).
Thanks dblaikie for the suggestion of adding a void use of the unused
variable instead.
llvm-svn: 252751
| -rw-r--r-- | llvm/lib/Target/ARM/ARMISelLowering.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/llvm/lib/Target/ARM/ARMISelLowering.cpp b/llvm/lib/Target/ARM/ARMISelLowering.cpp index 166d551..87bd9be 100644 --- a/llvm/lib/Target/ARM/ARMISelLowering.cpp +++ b/llvm/lib/Target/ARM/ARMISelLowering.cpp @@ -9141,7 +9141,9 @@ static SDValue PerformBFICombine(SDNode *N, SDValue From1 = ParseBFI(N, ToMask1, FromMask1); APInt ToMask2, FromMask2; - assert(From1 == ParseBFI(CombineBFI.getNode(), ToMask2, FromMask2)); + SDValue From2 = ParseBFI(CombineBFI.getNode(), ToMask2, FromMask2); + assert(From1 == From2); + (void)From2; // First, unlink CombineBFI. DCI.DAG.ReplaceAllUsesWith(CombineBFI, CombineBFI.getOperand(0)); |