This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [gold][aarch64]: Fixing gold pr/21491 - Errata workaround can produce broken images.
- From: Cary Coutant <ccoutant at gmail dot com>
- To: Han Shen <shenhan at google dot com>
- Cc: binutils <binutils at sourceware dot org>, Peter Smith <peter dot smith at linaro dot org>, Eric Christopher <echristo at gmail dot com>, Jing Yu <jingyu at google dot com>, Luis Lozano <llozano at google dot com>, Caroline Tice <cmtice at google dot com>, Rahul Chaudhry <rahulchaudhry at google dot com>
- Date: Wed, 12 Jul 2017 07:35:38 -0700
- Subject: Re: [gold][aarch64]: Fixing gold pr/21491 - Errata workaround can produce broken images.
- Authentication-results: sourceware.org; auth=none
- References: <CACkGtrgGO2ToZ1T-9kr7Uu2XnqVp8qhgH_N4=chaJOi__fZO0w@mail.gmail.com> <CACkGtrjAa0jyeqsrQ4txew1zy_2jUDRH2WfW2sm4CSo=-v-bsw@mail.gmail.com> <CAJimCsGwJbEyAyRuf++3nbmu0hWRxMA3hGcACsXDOv1ghjrHOA@mail.gmail.com> <CACkGtrj_yZ=7LLntAn8DifasVJmiLK-WyWEc8RoT_=K_VAo_QQ@mail.gmail.com> <CAJimCsGk=-rJqzv-M2e8FyDT74vHp9Oh-gpHYa_HUgC7JJE+Wg@mail.gmail.com> <CAJimCsHf7U1Kh1+aJ7F28acMKqP=ea9RSDK_2X8fTFQwVQaWgQ@mail.gmail.com>
I've backported Han's patch onto both the 2.28 and 2.29 branches.
-cary
On Mon, Jul 10, 2017 at 9:43 PM, Cary Coutant <ccoutant@gmail.com> wrote:
> 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