diff options
author | Kazu Hirata <kazu@google.com> | 2021-12-14 10:46:57 -0800 |
---|---|---|
committer | Kazu Hirata <kazu@google.com> | 2021-12-14 10:46:57 -0800 |
commit | ee99426c770605bd3eacc0d908a9b30e7f61dab9 (patch) | |
tree | ad3b02a17e29663434fe4226a86a0e48698cfa17 | |
parent | 6a44013b0e319979ce6ea9ead2a740564e434f76 (diff) | |
download | llvm-ee99426c770605bd3eacc0d908a9b30e7f61dab9.tar.gz |
[AArch64] Revise a warning fix
This patch revises the warning fix done in
a93b1792f1c8f7e2e7c931993110dc48f7ddba01. Specifically, it rolls the
MRI.getType call into the assert, thereby avoiding the named variable.
-rw-r--r-- | llvm/lib/Target/AArch64/GISel/AArch64PostLegalizerCombiner.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/llvm/lib/Target/AArch64/GISel/AArch64PostLegalizerCombiner.cpp b/llvm/lib/Target/AArch64/GISel/AArch64PostLegalizerCombiner.cpp index bbf6720985b2..3dec980a819a 100644 --- a/llvm/lib/Target/AArch64/GISel/AArch64PostLegalizerCombiner.cpp +++ b/llvm/lib/Target/AArch64/GISel/AArch64PostLegalizerCombiner.cpp @@ -312,9 +312,8 @@ static void applySplitStoreZero128(MachineInstr &MI, MachineRegisterInfo &MRI, GISelChangeObserver &Observer) { B.setInstrAndDebugLoc(MI); GStore &Store = cast<GStore>(MI); - LLT ValTy = MRI.getType(Store.getValueReg()); - assert(ValTy.isVector() && "Expected a vector store value"); - (void)ValTy; + assert(MRI.getType(Store.getValueReg()).isVector() && + "Expected a vector store value"); LLT NewTy = LLT::scalar(64); Register PtrReg = Store.getPointerReg(); auto Zero = B.buildConstant(NewTy, 0); |