On 12/05/13 17:47, Venkataramanan Kumar wrote:

+    # We support profiling for AArch64 linux target.
+    if { [istarget aarch64*-linux-*]
+ && ($test_what == "-p" || $test_what == "-pg") } {
+ return 1
      }

Can't this clause be removed completely now ?

We are not supporting profiling in "newlib". So it is still disabled
for aarch64-none-elf targets which uses newlib.

Indeed, reading the code for check_profiling_available it appears to me that we can remove this clause entirely and handle newlib in the same manner as all of the other targets that preclude profiling support on newlib.



Patch1
---------

gcc/ChangeLog
---------------------

2013-05-12  Venkataramanan Kumar  <venkataramanan.ku...@linaro.org>

        *  config/aarch64/aarch64.c (aarch64_return_addr): Handle returning
           address from a frame.

Patch2
----------
2013-05-12  Venkataramanan Kumar  <venkataramanan.ku...@linaro.org>

         * config/aarch64/aarch64.h (MCOUNT_NAME): Define.
           (NO_PROFILE_COUNTERS): Likewise.
           (PROFILE_HOOK): Likewise.
           (FUNCTION_PROFILER): Likewise.
        *  config/aarch64/aarch64.c (aarch64_function_profiler): Remove.
           (aarch64_frame_pointer_required): Enable frame pointer when 
profiling.

The usual convention on this list is one patch per email.
/Marcus


Reply via email to