[gold][aarch64]: Fixing gold pr/21491 - Errata workaround can produce broken images.

Cary Coutant ccoutant@gmail.com
Tue Jul 11 04:43:00 GMT 2017


On Mon, Jul 10, 2017 at 9:41 PM, Cary Coutant <ccoutant@gmail.com> wrote:
>>     gold/ChangeLog:
>>
>>       2017-07-06  Han Shen  <shenhan@google.com>
>>         * aarch64.cc (Erratum_stub::invalidate_erratum_stub): New method.
>>         (Erratum_stub::is_invalidated_erratum_stub): New method.
>>         (Stub_table::relocate_reloc_stub): Renamed from "relocate_stub".
>>         (Stub_table::relocate_reloc_stubs): Renamed from "relocate_stubs".
>>         (Stub_table::relocate_erratum_stub): New method.
>>         (AArch64_relobj::fix_errata_and_relocate_erratum_stubs): Renamed from
>>         "fix_errata".
>>         (Target_aarch64::relocate_reloc_stub): Renamed from "relocate_stub".
>
> +  switch (estub->type())
> +  {
> +    case ST_E_843419:
> +    case ST_E_835769:
>
> The braces for the switch should be indented two spaces, but the case
> labels remain where they are -- even with the indented braces.
>
> +          // stub. Note, when erratumf ix is skipped (either because we
>
> s/f ix/ fix/
>
> OK with those changes. Thanks!

Oops, one more thing... Please add "PR gold/21491" at the top of the
ChangeLog entry and commit message.

-cary



More information about the Binutils mailing list