[gold][aarch64] Patch for erratum-835769
Han Shen
shenhan@google.com
Fri Jun 12 21:38:00 GMT 2015
Hi Cary, thanks. All addressed and CL submitted.
On Fri, Jun 12, 2015 at 12:21 PM, Cary Coutant <ccoutant@gmail.com> wrote:
>> 2015-05-12 Han Shen <shenhan@google.com>
>>
>> * aarch64.cc (AArch64_insn_utilities::BYTES_PER_INSN): Move
>> defintion outside class definition.
>> (AArch64_insn_utilities::AARCH64_ZR): New static constant.
>> (AArch64_insn_utilities::aarch64_op31): New member.
>> (AArch64_insn_utilities::aarch64_ra): New member.
>> (AArch64_insn_utilities::aarch64_mac): New member.
>> (AArch64_insn_utilities::aarch64_mlxl): New member.
>> (ST_E_835769): New global enum member.
>> (Stub_table::relocate_stubs): Add 835769 handler.
>> (Stub_template_repertoire::Stub_template_repertoire): Install new
>> stub type.
>> (AArch64_relobj::scan_errata): This func is renamed from
>> scan_erratum_843419.
>> (AArch64_relobj::do_count_local_symbols): Add 835769 handler.
>> (AArch64_relobj::do_relocate_sections): Add 835769 handler.
>> (AArch64_relobj::scan_sections_for_stubs): Add 835769 handler.
>> (Target_aarch64::scan_erratum_835769_span): New method.
>> (Target_aarch64::create_erratum_stub): New method.
>> (Target_aarch64::is_erratum_835769_sequence): New method.
>> (Target_aarch64::scan_erratum_843419_sequence): Move part of the
>> code into create_erratum_stub.
>> * options.h (fix_cortex_a53_835769): New option.
>
> This is OK. Thanks!
>
> A few more typos:
>
> + /* Exclude MUL instructions which are encoded as a multiple accumulate
>
> "multiply-accumulate"
Done
>
> +// scanning the last few insns of a page, whereas for 835769, we need
> scan every
> +// insns.
>
> "we need to scan every insn."
Done
>
> + N_("(AArch64 only) Fix binaries for Cortex-A53 errata 843419."),
> + N_("(AArch64 only) Do not fix binaries for Cortex-A53 "
> "errata 843419."));
>
> + DEFINE_bool(fix_cortex_a53_835769, options::TWO_DASHES, '\0', false,
> + N_("(AArch64 only) Fix binaries for Cortex-A53 errata 835769."),
> + N_("(AArch64 only) Do not fix binaries for Cortex-A53 "
> + "errata 835769."));
>
> I think these help strings should use the singular form "erratum".
> Although you're scanning and fixing multiple instances of each, the
> message is actually referring to the erratum itself.
>
> The phrasing "fix binaries for" also doesn't sound right. "Scan for"
> worked, but not "fix for". How about just "Fix Cortex-A53 erratum
> XXXXXX"?
Use string "Fix Cortex-A53 erratum XXXXXX" in help string.
>
> -cary
--
Han Shen
More information about the Binutils
mailing list