Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 17 Jan 2017 20:46:51 +0100
From:      Roman Divacky <rdivacky@vlakno.cz>
To:        Justin Hibbits <chmeeedalf@gmail.com>
Cc:        Mark Millard <markmi@dsl-only.net>, FreeBSD Toolchain <freebsd-toolchain@freebsd.org>, FreeBSD PowerPC ML <freebsd-ppc@freebsd.org>
Subject:   Re: clang 3.9.0 buildkernel on old powerpc64's vs. trying to build hwpmc_e500.o and the like. . .
Message-ID:  <20170117194651.GA88907@vlakno.cz>
In-Reply-To: <A4A2D39B-8101-4E96-9107-9F2E6058BF49@gmail.com>
References:  <300CB7A2-34BB-407F-B2E9-D263B119A989@dsl-only.net> <20161205161904.GA7889@vlakno.cz> <A4A2D39B-8101-4E96-9107-9F2E6058BF49@gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
Are you sure? My coy of PowerISA lists the numbers that I used? What makes you
think it should be shifted by one bit?

On Mon, Jan 16, 2017 at 08:45:58PM -0600, Justin Hibbits wrote:
> The patch is incorrect, the 'xo' values are off by one bit (inline  
> change):
> 
> 
> On Dec 5, 2016, at 10:19 AM, Roman Divacky wrote:
> 
> > Can you try this patch?
> >
> > Index: llvm/lib/Target/PowerPC/PPCInstrInfo.td
> > ===================================================================
> > --- llvm/lib/Target/PowerPC/PPCInstrInfo.td	(revision 288675)
> > +++ llvm/lib/Target/PowerPC/PPCInstrInfo.td	(working copy)
> > @@ -2373,6 +2373,12 @@
> > def MFTB : XFXForm_1<31, 371, (outs gprc:$RT), (ins i32imm:$SPR),
> >                      "mftb $RT, $SPR", IIC_SprMFTB>;
> >
> > +def MFPMR : XFXForm_1<31, 334, (outs gprc:$RT), (ins i32imm:$PMRN),
> 
> 334 should be 167
> 
> > +                     "mfpmr $RT, $PMRN", IIC_IntGeneral>;
> > +
> > +def MTPMR : XFXForm_1<31, 462, (outs), (ins i32imm:$PMRN, gprc:$RS),
> > +                     "mtpmr $PMRN, $RS", IIC_IntGeneral>;
> 
> 462 should be 231.
> 
> > +
> > // A pseudo-instruction used to implement the read of the 64-bit  
> > cycle counter
> > // on a 32-bit target.
> > let hasSideEffects = 1, usesCustomInserter = 1 in
> >
> 
> I'll have a patch ready for LLVM review within a week or so, including  
> some level of scheduling.
> 
> - Justin



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