[llvm] a93b179 - [AArch64] Fix a warning

[David Blaikie via llvm-commits]

Maybe it'd be better to roll the MRI.getType call into the assert and
avoiding the named variable?

On Fri, Dec 10, 2021 at 8:33 AM Kazu Hirata via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

>
> Author: Kazu Hirata
> Date: 2021-12-10T08:33:07-08:00
> New Revision: a93b1792f1c8f7e2e7c931993110dc48f7ddba01
>
> URL:
> https://github.com/llvm/llvm-project/commit/a93b1792f1c8f7e2e7c931993110dc48f7ddba01
> DIFF:
> https://github.com/llvm/llvm-project/commit/a93b1792f1c8f7e2e7c931993110dc48f7ddba01.diff
>
> LOG: [AArch64] Fix a warning
>
> This patch fixes:
>
>   llvm/lib/Target/AArch64/GISel/AArch64PostLegalizerCombiner.cpp:315:7:
>   error: unused variable 'ValTy' [-Werror,-Wunused-variable]
>
> Added:
>
>
> Modified:
>     llvm/lib/Target/AArch64/GISel/AArch64PostLegalizerCombiner.cpp
>
> Removed:
>
>
>
>
> ################################################################################
> diff  --git
> a/llvm/lib/Target/AArch64/GISel/AArch64PostLegalizerCombiner.cpp
> b/llvm/lib/Target/AArch64/GISel/AArch64PostLegalizerCombiner.cpp
> index 5316a01b5ed94..bbf6720985b2c 100644
> --- a/llvm/lib/Target/AArch64/GISel/AArch64PostLegalizerCombiner.cpp
> +++ b/llvm/lib/Target/AArch64/GISel/AArch64PostLegalizerCombiner.cpp
> @@ -314,6 +314,7 @@ static void applySplitStoreZero128(MachineInstr &MI,
> MachineRegisterInfo &MRI,
>    GStore &Store = cast<GStore>(MI);
>    LLT ValTy = MRI.getType(Store.getValueReg());
>    assert(ValTy.isVector() && "Expected a vector store value");
> +  (void)ValTy;
>    LLT NewTy = LLT::scalar(64);
>    Register PtrReg = Store.getPointerReg();
>    auto Zero = B.buildConstant(NewTy, 0);
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211213/4297a0db/attachment.html>