From 9184ef8a92371746d0f36580adf2ad6845f1ced3 Mon Sep 17 00:00:00 2001 From: Nelson Chu Date: Wed, 19 Aug 2020 11:49:24 +0800 Subject: [PATCH] RISC-V: Minor cleanup and typos when merging elf attributes. bfd/ * elfnn-riscv.c (riscv_i_or_e_p): Minor cleanup for warnings/errors. (riscv_merge_std_ext): Likewise. (riscv_merge_arch_attr_info): Likewise. (riscv_merge_attributes): Likewise and fix comment typos. ld/ * testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-01.d: Remove the useless `warnings` keywords. * testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-02.d: Likewise. * testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-03.d: Likewise. * testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-04.d: Likewise. * testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-05.d: Likewise. * testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-06.d: Likewise. --- bfd/ChangeLog | 7 ++++++ bfd/elfnn-riscv.c | 24 +++++++++---------- ld/ChangeLog | 10 ++++++++ .../attr-merge-priv-spec-failed-01.d | 4 ++-- .../attr-merge-priv-spec-failed-02.d | 4 ++-- .../attr-merge-priv-spec-failed-03.d | 4 ++-- .../attr-merge-priv-spec-failed-04.d | 4 ++-- .../attr-merge-priv-spec-failed-05.d | 4 ++-- .../attr-merge-priv-spec-failed-06.d | 4 ++-- 9 files changed, 41 insertions(+), 24 deletions(-) diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 3bf4d293889..2dc5b542fb1 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,10 @@ +2020-09-03 Nelson Chu + + * elfnn-riscv.c (riscv_i_or_e_p): Minor cleanup for warnings/errors. + (riscv_merge_std_ext): Likewise. + (riscv_merge_arch_attr_info): Likewise. + (riscv_merge_attributes): Likewise and fix comment typos. + 2020-09-03 Nelson Chu * elfnn-riscv.c (riscv_version_mismatch): Change the return type diff --git a/bfd/elfnn-riscv.c b/bfd/elfnn-riscv.c index 83d4e63ceb4..c08827873cd 100644 --- a/bfd/elfnn-riscv.c +++ b/bfd/elfnn-riscv.c @@ -2667,8 +2667,8 @@ riscv_i_or_e_p (bfd *ibfd, && (strcasecmp (subset->name, "i") != 0)) { _bfd_error_handler - (_("error: %pB: corrupted ISA string '%s'. " - "First letter should be 'i' or 'e' but got '%s'."), + (_("error: %pB: corrupted ISA string '%s'. " + "First letter should be 'i' or 'e' but got '%s'"), ibfd, arch, subset->name); return FALSE; } @@ -2711,7 +2711,7 @@ riscv_merge_std_ext (bfd *ibfd, { /* TODO: We might allow merge 'i' with 'e'. */ _bfd_error_handler - (_("error: %pB: Mis-matched ISA string to merge '%s' and '%s'."), + (_("error: %pB: mis-matched ISA string to merge '%s' and '%s'"), ibfd, in->name, out->name); return FALSE; } @@ -2897,7 +2897,7 @@ riscv_merge_arch_attr_info (bfd *ibfd, char *in_arch, char *out_arch) { _bfd_error_handler (_("error: %pB: ISA string of input (%s) doesn't match " - "output (%s)."), ibfd, in_arch, out_arch); + "output (%s)"), ibfd, in_arch, out_arch); return NULL; } @@ -2917,15 +2917,15 @@ riscv_merge_arch_attr_info (bfd *ibfd, char *in_arch, char *out_arch) { _bfd_error_handler (_("error: %pB: XLEN of input (%u) doesn't match " - "output (%u)."), ibfd, xlen_in, xlen_out); + "output (%u)"), ibfd, xlen_in, xlen_out); return NULL; } if (xlen_in != ARCH_SIZE) { _bfd_error_handler - (_("error: %pB: Unsupported XLEN (%u), you might be " - "using wrong emulation."), ibfd, xlen_in); + (_("error: %pB: unsupported XLEN (%u), you might be " + "using wrong emulation"), ibfd, xlen_in); return NULL; } @@ -3039,7 +3039,7 @@ riscv_merge_attributes (bfd *ibfd, struct bfd_link_info *info) { _bfd_error_handler (_("warning: %pB use privilege spec version %u.%u.%u but " - "the output use version %u.%u.%u."), + "the output use version %u.%u.%u"), ibfd, in_attr[Tag_a].i, in_attr[Tag_b].i, @@ -3048,7 +3048,7 @@ riscv_merge_attributes (bfd *ibfd, struct bfd_link_info *info) out_attr[Tag_b].i, out_attr[Tag_c].i); - /* The priv spec v1.9.1 can be linked with other spec + /* The priv spec v1.9.1 can not be linked with other spec versions since the conflicts. We plan to drop the v1.9.1 in a year or two, so this confict should be removed in the future. */ @@ -3057,10 +3057,10 @@ riscv_merge_attributes (bfd *ibfd, struct bfd_link_info *info) { _bfd_error_handler (_("warning: privilege spec version 1.9.1 can not be " - "linked with other spec versions.")); + "linked with other spec versions")); } - /* Update the output priv attributes to the newest. */ + /* Update the output priv spec to the newest one. */ if (in_priv_spec > out_priv_spec) { out_attr[Tag_a].i = in_attr[Tag_a].i; @@ -3085,7 +3085,7 @@ riscv_merge_attributes (bfd *ibfd, struct bfd_link_info *info) { _bfd_error_handler (_("error: %pB use %u-byte stack aligned but the output " - "use %u-byte stack aligned."), + "use %u-byte stack aligned"), ibfd, in_attr[i].i, out_attr[i].i); result = FALSE; } diff --git a/ld/ChangeLog b/ld/ChangeLog index 185f37b031f..261390a94f8 100644 --- a/ld/ChangeLog +++ b/ld/ChangeLog @@ -1,3 +1,13 @@ +2020-09-03 Nelson Chu + + * testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-01.d: Remove + the useless `warnings` keywords. + * testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-02.d: Likewise. + * testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-03.d: Likewise. + * testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-04.d: Likewise. + * testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-05.d: Likewise. + * testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-06.d: Likewise. + 2020-09-03 Nelson Chu * testsuite/ld-riscv-elf/attr-merge-arch-failed-01.d: Update the diff --git a/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-01.d b/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-01.d index 0d5d6dc4a02..f3e2f2ae070 100644 --- a/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-01.d +++ b/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-01.d @@ -2,8 +2,8 @@ #source: attr-merge-priv-spec-c.s #as: #ld: -r -#warning: .*warning: .*use privilege spec version 1.11.0 but the output use version 1.9.1. -#warning: .*warning: .*privilege spec version 1.9.1 can not be linked with other spec versions. +#warning: .*use privilege spec version 1.11.0 but the output use version 1.9.1 +#warning: .*privilege spec version 1.9.1 can not be linked with other spec versions #readelf: -A Attribute Section: riscv diff --git a/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-02.d b/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-02.d index f0f75b2ba5a..9e98809b533 100644 --- a/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-02.d +++ b/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-02.d @@ -2,8 +2,8 @@ #source: attr-merge-priv-spec-a.s #as: #ld: -r -#warning: .*warning: .*use privilege spec version 1.9.1 but the output use version 1.11.0. -#warning: .*warning: .*privilege spec version 1.9.1 can not be linked with other spec versions. +#warning: .*use privilege spec version 1.9.1 but the output use version 1.11.0 +#warning: .*privilege spec version 1.9.1 can not be linked with other spec versions #readelf: -A Attribute Section: riscv diff --git a/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-03.d b/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-03.d index af5155271e3..05dffda2471 100644 --- a/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-03.d +++ b/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-03.d @@ -3,8 +3,8 @@ #source: attr-merge-priv-spec-c.s #as: #ld: -r -#warning: .*warning: .*use privilege spec version 1.11.0 but the output use version 1.9.1. -#warning: .*warning: .*privilege spec version 1.9.1 can not be linked with other spec versions. +#warning: .*use privilege spec version 1.11.0 but the output use version 1.9.1 +#warning: .*privilege spec version 1.9.1 can not be linked with other spec versions #readelf: -A Attribute Section: riscv diff --git a/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-04.d b/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-04.d index 23288070861..ff10f3f65c8 100644 --- a/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-04.d +++ b/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-04.d @@ -3,8 +3,8 @@ #source: attr-merge-priv-spec-c.s #as: #ld: -r -#warning: .*warning: .*use privilege spec version 1.11.0 but the output use version 1.9.1. -#warning: .*warning: .*privilege spec version 1.9.1 can not be linked with other spec versions. +#warning: .*use privilege spec version 1.11.0 but the output use version 1.9.1 +#warning: .*privilege spec version 1.9.1 can not be linked with other spec versions #readelf: -A Attribute Section: riscv diff --git a/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-05.d b/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-05.d index cabaab68f54..7c1cc56b6fe 100644 --- a/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-05.d +++ b/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-05.d @@ -3,8 +3,8 @@ #source: attr-merge-priv-spec-a.s #as: #ld: -r -#warning: .*warning: .*use privilege spec version 1.9.1 but the output use version 1.11.0. -#warning: .*warning: .*privilege spec version 1.9.1 can not be linked with other spec versions. +#warning: .*use privilege spec version 1.9.1 but the output use version 1.11.0 +#warning: .*privilege spec version 1.9.1 can not be linked with other spec versions #readelf: -A Attribute Section: riscv diff --git a/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-06.d b/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-06.d index e7747488880..1a81538fe9d 100644 --- a/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-06.d +++ b/ld/testsuite/ld-riscv-elf/attr-merge-priv-spec-failed-06.d @@ -3,8 +3,8 @@ #source: attr-merge-priv-spec-a.s #as: #ld: -r -#warning: .*warning: .*use privilege spec version 1.9.1 but the output use version 1.11.0. -#warning: .*warning: .*privilege spec version 1.9.1 can not be linked with other spec versions. +#warning: .*use privilege spec version 1.9.1 but the output use version 1.11.0 +#warning: .*privilege spec version 1.9.1 can not be linked with other spec versions #readelf: -A Attribute Section: riscv -- 2.30.2