arm: PR 29739 Fix typo where '; ' should not have been replaced with '@'

Message ID 20221102172923.4009281-1-christophe.lyon@arm.com
State Repeat Merge
Headers
Series arm: PR 29739 Fix typo where '; ' should not have been replaced with '@' |

Checks

Context Check Description
snail/binutils-gdb-check warning Git am fail log

Commit Message

Christophe Lyon Nov. 2, 2022, 5:29 p.m. UTC
  ';' does not always indicate the start of a comment, and commit
8cb6e17571f3fb66ccd4fa19f881602542cd06fc incorrectly replaced 3
instances of ';' with '@' in expected diagnostics, leading to tests
failures.

This patch restores the original ';' as needed in these testcases.

Fixes bug: https://sourceware.org/bugzilla/show_bug.cgi?id=29739
---
 ld/testsuite/ld-arm/attr-merge-wchar-24.d          | 2 +-
 ld/testsuite/ld-arm/attr-merge-wchar-42.d          | 2 +-
 ld/testsuite/ld-arm/stm32l4xx-cannot-fix-far-ldm.d | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)
  

Comments

Nick Clifton Nov. 2, 2022, 5:51 p.m. UTC | #1
Hi Christophe,

> ';' does not always indicate the start of a comment, and commit
> 8cb6e17571f3fb66ccd4fa19f881602542cd06fc incorrectly replaced 3
> instances of ';' with '@' in expected diagnostics, leading to tests
> failures. >
> This patch restores the original ';' as needed in these testcases.
> 
> Fixes bug: https://sourceware.org/bugzilla/show_bug.cgi?id=29739
> 
Patch approved - please apply.

Cheers
   Nick
  

Patch

diff --git a/ld/testsuite/ld-arm/attr-merge-wchar-24.d b/ld/testsuite/ld-arm/attr-merge-wchar-24.d
index 92a26ab505b..46d6c66f353 100644
--- a/ld/testsuite/ld-arm/attr-merge-wchar-24.d
+++ b/ld/testsuite/ld-arm/attr-merge-wchar-24.d
@@ -2,4 +2,4 @@ 
 #source: attr-merge-wchar-4.s
 #as:
 #ld: -r
-#warning: warning: .* uses 4-byte wchar_t yet the output is to use 2-byte wchar_t@ use of wchar_t values across objects may fail
+#warning: warning: .* uses 4-byte wchar_t yet the output is to use 2-byte wchar_t; use of wchar_t values across objects may fail
diff --git a/ld/testsuite/ld-arm/attr-merge-wchar-42.d b/ld/testsuite/ld-arm/attr-merge-wchar-42.d
index 274cdc84fbc..c2aca5e67cc 100644
--- a/ld/testsuite/ld-arm/attr-merge-wchar-42.d
+++ b/ld/testsuite/ld-arm/attr-merge-wchar-42.d
@@ -2,4 +2,4 @@ 
 #source: attr-merge-wchar-2.s
 #as:
 #ld: -r
-#warning: warning: .* uses 2-byte wchar_t yet the output is to use 4-byte wchar_t@ use of wchar_t values across objects may fail
+#warning: warning: .* uses 2-byte wchar_t yet the output is to use 4-byte wchar_t; use of wchar_t values across objects may fail
diff --git a/ld/testsuite/ld-arm/stm32l4xx-cannot-fix-far-ldm.d b/ld/testsuite/ld-arm/stm32l4xx-cannot-fix-far-ldm.d
index 13b28385bf3..8e785c611a9 100644
--- a/ld/testsuite/ld-arm/stm32l4xx-cannot-fix-far-ldm.d
+++ b/ld/testsuite/ld-arm/stm32l4xx-cannot-fix-far-ldm.d
@@ -3,7 +3,7 @@ 
 #ld:-EL --fix-stm32l4xx-629360 -Ttext=0x80000
 #objdump: -dr --prefix-addresses --show-raw-insn
 #name: STM32L4XX erratum : LDM cannot be patched when LDM is too far from veneer section
-#warning: .*cannot create STM32L4XX veneer@ jump out of range by 24 bytes; cannot encode branch instruction.*
+#warning: .*cannot create STM32L4XX veneer; jump out of range by 24 bytes; cannot encode branch instruction.*
 
 # Test the `LDM*' instructions when too far from the veneer section
 # They cannot, thus should not, be patched