Skip to contentSkip to conversation

gcc: arc-2017.03-rc2: Fix "unrecognized supposed constant" error

PATCH New Alexey Brodkin 2 participants 4 messages May 15, 2017 – May 16, 2017

lore
Share & tools
lore

Alexey Brodkin

author
#0 lore raw

PATCH 1/1 gcc: arc-2017.03-rc2: Fix "unrecognized supposed constant" error

This fixes the following problem:
------------------------------------>8-------------------------------- arc-linux-gcc -c -Os -fPIC iso9660.i
iso9660.c: In function 'strip_trail':
iso9660.c:155:1: error: unrecognized supposed constant
}
^
(unspec:SI [

        (symbol_ref:SI ("*.LANCHOR1") [flags 0x182])
    ] ARC_UNSPEC_GOTOFFPC)

iso9660.c:155:1: internal compiler error: in arc_legitimate_constant_p, at config/arc/arc.c:6028 ------------------------------------>8--------------------------------

Found by Buildroot autobuilder [1].

The fix [2] is in arc-2017.03 development branch of ARC GCC and once it becomes a part the next release of ARC tools this should be removed from Buildroot.

[1] http://autobuild.buildroot.net/results/c90/c909e8c397ab972b6aa4d370572cad4fae284d00/build-end.log
[2] https://github.com/foss-for-synopsys-dwc-arc-processors/gcc/commit/139fed9d29ab935b3bc5159c0bdf7b8b8a39442d

Signed-off-by Alexey Brodkin <[email protected]>
---
 ...-return-false-for-any-PIC-related-unspecs.patch | 58 ++++++++++++++++++++++
 1 file changed, 58 insertions(+)
 create mode 100644 package/gcc/arc-2017.03-rc2/0001-FIX-PIC-return-false-for-any-PIC-related-unspecs.patch
package/gcc/arc-2017.03-rc2/0001-FIX-PIC-return-false-for-any-PIC-related-unspecs.patch new file +58 -0
new file mode 100644
index 0000000000..ea9708240e
@@ -0,0 +1,58 @@
+From 139fed9d29ab935b3bc5159c0bdf7b8b8a39442d Mon Sep 17 00:00:00 2001
+From: Claudiu Zissulescu <[email protected]>
+Date: Mon, 15 May 2017 05:05:27 -0400
+Subject: [PATCH] [FIX] PIC: return false for any PIC related  unspecs
+
+---
+ gcc/config/arc/arc.c                        |  9 ++++-----
+ gcc/testsuite/gcc.target/arc/pr9001191897.c | 10 ++++++++++
+ 2 files changed, 14 insertions(+), 5 deletions(-)
+ create mode 100644 gcc/testsuite/gcc.target/arc/pr9001191897.c
+
+diff --git a/gcc/config/arc/arc.c b/gcc/config/arc/arc.c
+index 3d53a667d37f..b00126f4d2c4 100644
+--- a/gcc/config/arc/arc.c
++++ b/gcc/config/arc/arc.c
+@@ -6056,11 +6056,8 @@ arc_legitimate_constant_p (machine_mode mode, rtx x)
+       return true;
+ 
+     case NEG:
+-      /* Assembler does not understand -(@label at gotoff).  Also, we do
+-	 not print such pic address constant.  */
+-      if (GET_CODE (XEXP (x, 0)) == UNSPEC)
+-	return false;
+       return arc_legitimate_constant_p (mode, XEXP (x, 0));
++
+     case PLUS:
+     case MINUS:
+       {
+@@ -6090,7 +6087,9 @@ arc_legitimate_constant_p (machine_mode mode, rtx x)
+ 	case UNSPEC_TLS_IE:
+ 	  return true;
+ 	default:
+-	  break;
++	  /* Any other unspec ending here are pic related, hence the above
++	     constant pic address checking returned false.  */
++	  return false;
+ 	}
+       /* Fall through.  */
+ 
+diff --git a/gcc/testsuite/gcc.target/arc/pr9001191897.c b/gcc/testsuite/gcc.target/arc/pr9001191897.c
+new file mode 100644
+index 000000000000..2b9e1689803f
+--- /dev/null
++++ b/gcc/testsuite/gcc.target/arc/pr9001191897.c
+@@ -0,0 +1,10 @@
++/* { dg-do compile } */
++/* { dg-skip-if "" { ! { clmcpu } } } */
++/* { dg-options "-mcpu=arch2 -Os -fpic -mno-sdata -mno-indexed-loads -w" } */
++a;
++c() {
++  static char b[25];
++  for (; a >= 0; a--)
++    if (b[a])
++      b[a] = '\0';
++}
+-- 
+2.7.4
+
Signature · 1 line
2.11.0
#1 lore raw

PATCH 1/1 gcc: arc-2017.03-rc2: Fix "unrecognized supposed constant" error

Quoted 15 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Alexey" == Alexey Brodkin <[email protected]> writes:

This fixes the following problem:
arc-linux-gcc -c -Os -fPIC iso9660.i iso9660.c: In function 'strip_trail':
iso9660.c:155:1: error: unrecognized supposed constant
}
^
(unspec:SI [

        (symbol_ref:SI ("*.LANCHOR1") [flags 0x182])
    ] ARC_UNSPEC_GOTOFFPC)

iso9660.c:155:1: internal compiler error: in arc_legitimate_constant_p, at config/arc/arc.c:6028

Found by Buildroot autobuilder [1].

Quoted 7 lines

The fix [2] is in arc-2017.03 development branch of ARC GCC and once it becomes a part the next release of ARC tools this should be removed from Buildroot.

[1] http://autobuild.buildroot.net/results/c90/c909e8c397ab972b6aa4d370572cad4fae284d00/build-end.log
[2]
https://github.com/foss-for-synopsys-dwc-arc-processors/gcc/commit/139fed9d29ab935b3bc5159c0bdf7b8b8a39442d

Committed, thanks.

I just noticed that we still have a patch from arc-2016.09. Can you confirm that this isn't needed any more?

https://git.buildroot.org/buildroot/tree/package/gcc/arc-2016.09-release/895-arc-define-_REENTRANT-when-pthread-is-passed.patch

Signature · 1 line
Bye, Peter Korsgaard
author
#2 lore raw

PATCH 1/1 gcc: arc-2017.03-rc2: Fix "unrecognized supposed constant" error

Hi Peter,

Quoted 32 lines · Peter Korsgaard
Quoted 6 lines
Quoted 6 lines
Quoted 6 lines
Quoted 6 lines
Quoted 6 lines

"Alexey" == Alexey Brodkin <[email protected]> writes:

?> This fixes the following problem:
?> arc-linux-gcc -c -Os -fPIC iso9660.i
?> iso9660.c: In function 'strip_trail':
?> iso9660.c:155:1: error: unrecognized supposed constant
?>??}
?>??^
?> (unspec:SI [
?>?????????(symbol_ref:SI ("*.LANCHOR1") [flags 0x182])
?>?????] ARC_UNSPEC_GOTOFFPC)
?> iso9660.c:155:1: internal compiler error: in arc_legitimate_constant_p, at config/arc/arc.c:6028

?> Found by Buildroot autobuilder [1].

?> The fix [2] is in arc-2017.03 development branch of ARC GCC and once it ?> becomes a part the next release of ARC tools this should be removed ?> from Buildroot.

?> [1] https://urldefense.proofpoint.com/v2/url?u=http-3A__autobuild.buildroot.net_results_c90_c909e8c397ab972b6aa4d370572cad4fae284d00_build-2Dend.
log&d=DwIBAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=lqdeeSSEes0GFDDl656eViXO7breS55ytWkhpk5R81I&m=eyEAZfBkQhTd80xpTFI0NPKpz57Aa8duF3Uu1L_JgXM&s=PiWGi8QYaMIbOlEL BukiJt2-WQi9IqfQlPFkxgNqbE8&e=?
?> [2]
?> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_foss-2Dfor-2Dsynopsys-2Ddwc-2Darc-2Dprocessors_gcc_commit_139fed9d29ab935b3bc5159
c0bdf7b8b8a39442d&d=DwIBAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=lqdeeSSEes0GFDDl656eViXO7breS55ytWkhpk5R81I&m=eyEAZfBkQhTd80xpTFI0NPKpz57Aa8duF3Uu1L_JgXM&s=OK r0FRgbINPdN6fRMnB18MICruG3PJVXVDpOjxISyVk&e=?

Quoted 1 line

Committed, thanks.

Thanks for that.

Quoted 2 lines

I just noticed that we still have a patch from arc-2016.09. Can you confirm that this isn't needed any more?

No this is not needed any longer - was just a reminder of incomplete patch that bumped ARC tools version.

Care to remove this or you'd prefer a patch from my on that regard?

-Alexey

#3 lore raw

PATCH 1/1 gcc: arc-2017.03-rc2: Fix "unrecognized supposed constant" error

Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Alexey" == Alexey Brodkin <[email protected]> writes:

Hi,

Quoted 7 lines
Quoted 2 lines

I just noticed that we still have a patch from arc-2016.09. Can you confirm that this isn't needed any more?

No this is not needed any longer - was just a reminder of incomplete patch that bumped ARC tools version.

Care to remove this or you'd prefer a patch from my on that regard?

I can remove it, I just wanted to verify that it wasn't needed any more
- Thanks.

Signature · 1 line
Bye, Peter Korsgaard