On Thu, Sep 18, 2014 at 1:11 PM, Eric Christopher <echri...@gmail.com> wrote:
>
>
>
> On Wed, Sep 10, 2014 at 3:39 AM, Matthew Fortune <matthew.fort...@imgtec.com>
> wrote:
>>
>> > The patch is ok to commit. I think I made a typo as other analogous
>> > code contains '|='.
>> >
>> > Thanks for fixing this, Robert.
>>
>> The LRA part of this is committed as r215119.
>>
>> Eric: Is the test OK to commit for MIPS? I suggest removing the
>> dg-skip-if as the test is for successful compilation, not inspecting
>> the generated code.
>>
Agreed. Sorry for the delay.
-eric
>
>>
>> Thanks,
>> Matthew
>>
>> >
>> > >
>> > > 2014-09-09 Robert Suchanek <robert.sucha...@imgtec.com>
>> > >
>> > > gcc/testsuite/
>> > > * gcc.target/mips/20140909.c: New test.
>>
>> > > diff --git a/gcc/testsuite/gcc.target/mips/20140909.c
>> > b/gcc/testsuite/gcc.target/mips/20140909.c
>> > > new file mode 100644
>> > > index 0000000..dbdfb30
>> > > --- /dev/null
>> > > +++ b/gcc/testsuite/gcc.target/mips/20140909.c
>> > > @@ -0,0 +1,20 @@
>> > > +/* { dg-do compile } */
>> > > +/* { dg-skip-if "code quality test" { *-*-* } { "-O0" } { "" } } */
>> > > +
>> > > +NOMIPS16 int NoBarrier_AtomicIncrement(volatile int* ptr, int
>> > increment) {
>> > > + int temp, temp2;
>> > > + __asm__ __volatile__(".set push\n"
>> > > + ".set noreorder\n"
>> > > + "1:\n"
>> > > + "ll %0, 0(%3)\n"
>> > > + "addu %1, %0, %2\n"
>> > > + "sc %1, 0(%3)\n"
>> > > + "beqz %1, 1b\n"
>> > > + "addu %1, %0, %2\n"
>> > > + ".set pop\n"
>> > > + : "=&r" (temp), "=&r" (temp2)
>> > > + : "Ir" (increment), "r" (ptr)
>> > > + : "memory");
>> > > +
>> > > + return temp2;
>> > > +}
>> > >
>>
>
