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