This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH], Add PowerPC ISA 3.0 Atomic Memory Operation functions
Hi!
On Fri, Oct 06, 2017 at 02:03:43PM -0400, Michael Meissner wrote:
> > Do you really need the mr insns? Can't you express that in the
> > arguments? Perhaps using a union of __int128 with something that
> > is two long ints?
>
> My first attempt resulted in the compiler doing move directs to form the
> __int128 in the vector unit and then move directs back. So, I figured having
> two mr's was a simple price to pay.
>
> I do have thoughts to replace the two with a built-in (but keep amo.h and the
> names), and we can probably eliminate some of the moves.
It's so ugly, even if it doesn't cost much ;-)
But don't worry about it, certainly not if the plan is to expand it
as a builtin later.
Segher