[llvm] d2bd075 - Fix -Wunused-variable after D80515

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 8 18:29:27 PDT 2020


Could you describe the nature of the change in the subject line? Makes
it easier to filter on what is actually changing, rather than what
motivated the change (the warning). (I know this can seem at odds with
the usual advice to describe the purpose of a change moreso than the
mechanical details of a change (since the mechanical details can be
seen in the associated diff) - but "Remove unused variable
(-Wunused-variable in D80515)" probably covers all of it?)

On Fri, Jun 5, 2020 at 11:47 AM Fangrui Song via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
>
>
> Author: Fangrui Song
> Date: 2020-06-05T11:46:50-07:00
> New Revision: d2bd075e8d1f4544357c0ab9784ec4e88bd229a7
>
> URL: https://github.com/llvm/llvm-project/commit/d2bd075e8d1f4544357c0ab9784ec4e88bd229a7
> DIFF: https://github.com/llvm/llvm-project/commit/d2bd075e8d1f4544357c0ab9784ec4e88bd229a7.diff
>
> LOG: Fix -Wunused-variable after D80515
>
> Added:
>
>
> Modified:
>     llvm/lib/Target/ARM/ARMISelLowering.cpp
>
> Removed:
>
>
>
> ################################################################################
> diff  --git a/llvm/lib/Target/ARM/ARMISelLowering.cpp b/llvm/lib/Target/ARM/ARMISelLowering.cpp
> index 10e7137f4e80..e4805eedbda5 100644
> --- a/llvm/lib/Target/ARM/ARMISelLowering.cpp
> +++ b/llvm/lib/Target/ARM/ARMISelLowering.cpp
> @@ -14460,7 +14460,6 @@ static SDValue PerformVMOVNCombine(SDNode *N,
>  static SDValue PerformVQMOVNCombine(SDNode *N,
>                                      TargetLowering::DAGCombinerInfo &DCI) {
>    SDValue Op0 = N->getOperand(0);
> -  SDValue Op1 = N->getOperand(1);
>    unsigned IsTop = N->getConstantOperandVal(2);
>
>    unsigned NumElts = N->getValueType(0).getVectorNumElements();
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits


More information about the llvm-commits mailing list