PATCH: Add LWP support for upcoming AMD Orochi processor.

Jakub Jelinek jakub@redhat.com
Fri Nov 6 10:15:00 GMT 2009


On Thu, Nov 05, 2009 at 05:21:24PM +0100, Jakub Jelinek wrote:
> As the following patch proves, lwpintrin.h and the whole lwp support is
> quite severely broken.

I forgot:

5) you shouldn't provide DI insns for !TARGET_64BIT, I don't think they are
valid in 32-bit mode

6) for insns that you keep multiple versions of and don't use :P iterator,
you should probably use :SWI248 mode iterator instead of duplicating
the pattern 3 times.

Once the lwpval/lwpins stuff in lwpintrin.h is uncommented, you'll also need
to further adjust my testsuite patch - you want to actually test the
intrinsics that require immediate arguments, like they are tested in other
intrin headers.

And as Uros mentioned, I think the patch hasn't been acked by anyone, just
some random comments have been posted about the patch by various people,
but never a real review.

	Jakub



More information about the Gcc-patches mailing list