Date: Mon, 16 Jan 2017 20:45:58 -0600 From: Justin Hibbits <chmeeedalf@gmail.com> To: Roman Divacky <rdivacky@vlakno.cz> 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: <A4A2D39B-8101-4E96-9107-9F2E6058BF49@gmail.com> In-Reply-To: <20161205161904.GA7889@vlakno.cz> References: <300CB7A2-34BB-407F-B2E9-D263B119A989@dsl-only.net> <20161205161904.GA7889@vlakno.cz>
next in thread | previous in thread | raw e-mail | index | archive | help
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?A4A2D39B-8101-4E96-9107-9F2E6058BF49>