Re: [PATCH][PR target/79170] fix memcmp builtin expansion sequence for rs6000 target.

[Peter Bergner]

On 1/27/17 5:43 PM, Segher Boessenkool wrote:

On Fri, Jan 27, 2017 at 12:11:05PM -0600, Aaron Sawdey wrote:

+            addi 9,4,7
+            lwbrx 10,0,9
+            addi 9,5,7
+            lwbrx 9,0,9

It would be nice if this was

        li 9,7
        lwbrx 10,9,4
        lwbrx 9,9,5

Nicer still, we want the base address as the RA operand
and the offset as the RB operand, so like so:

        li 9,7
        lwbrx 10,4,9
        lwbrx 9,5,9

On some processors, it matters performance wise.

Peter