[llvm] r260773 - AMDGPU: Fix broken condition causing warning

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 16 14:25:22 PST 2016


Is there a missing test case here?

On Fri, Feb 12, 2016 at 4:36 PM, Matt Arsenault via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> Author: arsenm
> Date: Fri Feb 12 18:36:10 2016
> New Revision: 260773
>
> URL: http://llvm.org/viewvc/llvm-project?rev=260773&view=rev
> Log:
> AMDGPU: Fix broken condition causing warning
>
> Modified:
>     llvm/trunk/lib/Target/AMDGPU/SIISelLowering.cpp
>
> Modified: llvm/trunk/lib/Target/AMDGPU/SIISelLowering.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AMDGPU/SIISelLowering.cpp?rev=260773&r1=260772&r2=260773&view=diff
>
> ==============================================================================
> --- llvm/trunk/lib/Target/AMDGPU/SIISelLowering.cpp (original)
> +++ llvm/trunk/lib/Target/AMDGPU/SIISelLowering.cpp Fri Feb 12 18:36:10
> 2016
> @@ -1197,7 +1197,7 @@ SDValue SITargetLowering::LowerFrameInde
>  }
>
>  bool SITargetLowering::isCFIntrinsic(const SDNode *Intr) const {
> -  if (!Intr->getOpcode() == ISD::INTRINSIC_W_CHAIN)
> +  if (Intr->getOpcode() != ISD::INTRINSIC_W_CHAIN)
>      return false;
>
>    switch (cast<ConstantSDNode>(Intr->getOperand(1))->getZExtValue()) {
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://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/20160216/0849d963/attachment.html>


More information about the llvm-commits mailing list