This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RS6000] Fix PR66020, -mprofile-kernel related


On Wed, May 6, 2015 at 5:50 AM, Alan Modra <amodra@gmail.com> wrote:
> -mprofile-kernel changed recently to not save lr on the stack before
> calling _mcount.  This means a change is required in the _mcount
> used by one of the powerpc64 tests to grab function parameter
> registers.  While fixing that, I noticed that the asm defined the
> _mcount label, which is a bit rude;  A function in ELFv1 is supposed
> to be defined on an OPD entry.  Using an alias fixes this problem.
> Also, _mcount used r11, the static chain pointer.  That doesn't really
> hurt here, but I thought it was poor form to have such an example
> of _mcount tricks in the testsuite.  Finally, I made gparms volatile
> since it does change behind gcc's back.
>
> Bootstrapped and regression tested powerpc64-linux and
> powerpc64le-linux.
>
>         PR target/66020
>         * gcc.target/powerpc/ppc64-abi-2.c (my_mcount): Rewrite.
>         (gparms): Make volatile.  Remove trailing whitespace.

Okay.

Thanks, David


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]