[Patch/ARM] Skip private symbol when doing objdump

[Ramana Radhakrishnan]

On Tue, Feb 3, 2015 at 11:37 AM, Jiong Wang <jiong.wang@arm.com> wrote:
> __tagsym$$ is used as a prefix to tag particular symbols by the proprietary
> ARM toolchain. This patch gives higher priority to proper symbol names so
> that they are shown in the disassembly appropriately.
>
> before this patch, for "objdump -D", sometime we display
>
> __tagsym$$noinline:
>    insn0
>    insn1
>    ...
>
> __tagsym$$used:
>    data_content
>    ...
>
> after this patch, we display:
>
> foo:
>   insn0
>   insn1
>
> global_a:
>   data_content
>   ...
>
> this result make more sense for the user.
>
> pass native full binutils check
>
> ok for trunk?
>

This is OK - thanks and sorry about the delay in reviewing this.

Ramana

> 2015-02-03  Jiong Wang  <jiong.wang@arm.com>
>
> opcodes/
>   * arm-dis.c (arm_symbol_is_valid): Skip ARM private symbols.
>
> binutils/testsuite/
>   * binutils-all/arm/rvct_symbol.s: New testcase.
>   * binutils-all/arm/objdump.exp: Run it.