Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 28 Jun 2014 08:26:37 -0700
From:      Cy Schubert <Cy.Schubert@komquats.com>
To:        Dimitry Andric <dim@FreeBSD.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r268003 - in head/contrib/llvm: lib/Target/PowerPC  patches
Message-ID:  <201406281526.s5SFQcRZ013277@slippy.cwsent.com>
In-Reply-To: Message from Dimitry Andric <dim@FreeBSD.org> of "Sat, 28 Jun 2014 09:53:44 -0000." <201406280953.s5S9ri9e067583@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
In message <201406280953.s5S9ri9e067583@svn.freebsd.org>, Dimitry Andric 
writes
:
> Author: dim
> Date: Sat Jun 28 09:53:44 2014
> New Revision: 268003
> URL: http://svnweb.freebsd.org/changeset/base/268003
> 
> Log:
>   Fix breakage after r267981.
>   
>   Pointy hat to:	dim
>   MFC after:	3 days
>   X-MFC-With:	r267981
> 
> Modified:
>   head/contrib/llvm/lib/Target/PowerPC/PPCFastISel.cpp
>   head/contrib/llvm/patches/patch-r267981-llvm-r211435-fix-ppc-fctiduz.diff
> 
> Modified: head/contrib/llvm/lib/Target/PowerPC/PPCFastISel.cpp
> =============================================================================
> =
> --- head/contrib/llvm/lib/Target/PowerPC/PPCFastISel.cpp	Sat Jun 28 05:5
> 1:45 2014	(r268002)
> +++ head/contrib/llvm/lib/Target/PowerPC/PPCFastISel.cpp	Sat Jun 28 09:5
> 3:44 2014	(r268003)
> @@ -1027,7 +1027,7 @@ bool PPCFastISel::SelectFPToI(const Inst
>      return false;
>  
>    // If we don't have FCTIDUZ and we need it, punt to SelectionDAG.
> -  if (DstVT == MVT::i64 && !IsSigned && !PPCSubTarget->hasFPCVT())
> +  if (DstVT == MVT::i64 && !IsSigned && !PPCSubTarget.hasFPCVT())
>      return false;
>  
>    Value *Src = I->getOperand(0);
> 
> Modified: head/contrib/llvm/patches/patch-r267981-llvm-r211435-fix-ppc-fctidu
> z.diff
> =============================================================================
> =
> --- head/contrib/llvm/patches/patch-r267981-llvm-r211435-fix-ppc-fctiduz.diff
> 	Sat Jun 28 05:51:45 2014	(r268002)
> +++ head/contrib/llvm/patches/patch-r267981-llvm-r211435-fix-ppc-fctiduz.diff
> 	Sat Jun 28 09:53:44 2014	(r268003)
> @@ -32,7 +32,7 @@ Index: lib/Target/PowerPC/PPCFastISel.cp
>       return false;
>   
>  +  // If we don't have FCTIDUZ and we need it, punt to SelectionDAG.
> -+  if (DstVT == MVT::i64 && !IsSigned && !PPCSubTarget->hasFPCVT())
> ++  if (DstVT == MVT::i64 && !IsSigned && !PPCSubTarget.hasFPCVT())
>  +    return false;
>  +
>     Value *Src = I->getOperand(0);
> 
> 

Thanks.

Looks like they changed PPCSubTarget to a pointer in llvm trunk about five 
weeks ago. :~


-- 
Cheers,
Cy Schubert <Cy.Schubert@komquats.com>
FreeBSD UNIX:  <cy@FreeBSD.org>   Web:  http://www.FreeBSD.org

	The need of the many outweighs the greed of the few.





Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201406281526.s5SFQcRZ013277>