[llvm] 9f507bf - NFC: fix unused var warnings in no-assert builds

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 26 11:32:52 PDT 2020


Ideally, rolling expressions into an assert if they're otherwise unused
would be preferred. (done this in 9002db05a2f0d12681214fad3c96f8d497f2b852)

On Wed, Feb 5, 2020 at 9:27 AM Jordan Rupprecht via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

>
> Author: Jordan Rupprecht
> Date: 2020-02-05T09:26:59-08:00
> New Revision: 9f507bfd8d4761c619f21d8165b7e6e6e5bd09bd
>
> URL:
> https://github.com/llvm/llvm-project/commit/9f507bfd8d4761c619f21d8165b7e6e6e5bd09bd
> DIFF:
> https://github.com/llvm/llvm-project/commit/9f507bfd8d4761c619f21d8165b7e6e6e5bd09bd.diff
>
> LOG: NFC: fix unused var warnings in no-assert builds
>
> Added:
>
>
> Modified:
>     llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
>
> Removed:
>
>
>
>
> ################################################################################
> diff  --git a/llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
> b/llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
> index 6f2e1ae51cfc..3f4dce23ce43 100644
> --- a/llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
> +++ b/llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
> @@ -1981,6 +1981,8 @@ bool AMDGPULegalizerInfo::legalizeBuildVector(
>    LLT DstTy = MRI.getType(Dst);
>    const LLT S32 = LLT::scalar(32);
>    const LLT V2S16 = LLT::vector(2, 16);
> +  (void)DstTy;
> +  (void)V2S16;
>    assert(DstTy == V2S16);
>
>    Register Src0 = MI.getOperand(1).getReg();
>
>
>
> _______________________________________________
> 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/20200326/cd7181fe/attachment.html>


More information about the llvm-commits mailing list