public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH] Do not enable DT_INIT_ARRAY/DT_FINI_ARRAY on uclinuxfdpiceabi
@ 2021-08-12 15:29 Christophe Lyon
  2021-08-16 11:53 ` Christophe LYON
  0 siblings, 1 reply; 3+ messages in thread
From: Christophe Lyon @ 2021-08-12 15:29 UTC (permalink / raw)
  To: gcc-patches

Commit r12-1328 enabled DT_INIT_ARRAY/DT_FINI_ARRAY for all Linux
targets, but this does not work for arm-none-uclinuxfdpiceabi: it
makes all the execution tests fail.

This patch restores the original behavior for uclinuxfdpiceabi.

2021-08-12  Christophe Lyon  <christophe.lyon@foss.st.com>

	gcc/
	PR target/100896
	* config.gcc (gcc_cv_initfini_array): Leave undefined for
	uclinuxfdpiceabi targets.
---
 gcc/config.gcc | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/gcc/config.gcc b/gcc/config.gcc
index 93e2b3219b9..8c8d30ca934 100644
--- a/gcc/config.gcc
+++ b/gcc/config.gcc
@@ -851,8 +851,14 @@ case ${target} in
   tmake_file="${tmake_file} t-glibc"
   target_has_targetcm=yes
   target_has_targetdm=yes
-  # Linux targets always support .init_array.
-  gcc_cv_initfini_array=yes
+  case $target in
+    *-*-uclinuxfdpiceabi)
+      ;;
+    *)
+      # Linux targets always support .init_array.
+      gcc_cv_initfini_array=yes
+      ;;
+  esac
   ;;
 *-*-netbsd*)
   tm_p_file="${tm_p_file} netbsd-protos.h"
-- 
2.25.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Do not enable DT_INIT_ARRAY/DT_FINI_ARRAY on uclinuxfdpiceabi
  2021-08-12 15:29 [PATCH] Do not enable DT_INIT_ARRAY/DT_FINI_ARRAY on uclinuxfdpiceabi Christophe Lyon
@ 2021-08-16 11:53 ` Christophe LYON
  2021-08-17  9:39   ` Richard Sandiford
  0 siblings, 1 reply; 3+ messages in thread
From: Christophe LYON @ 2021-08-16 11:53 UTC (permalink / raw)
  To: gcc-patches; +Cc: hjl.tools, joseph

ping?


On 12/08/2021 17:29, Christophe Lyon via Gcc-patches wrote:
> Commit r12-1328 enabled DT_INIT_ARRAY/DT_FINI_ARRAY for all Linux
> targets, but this does not work for arm-none-uclinuxfdpiceabi: it
> makes all the execution tests fail.
>
> This patch restores the original behavior for uclinuxfdpiceabi.
>
> 2021-08-12  Christophe Lyon  <christophe.lyon@foss.st.com>
>
> 	gcc/
> 	PR target/100896
> 	* config.gcc (gcc_cv_initfini_array): Leave undefined for
> 	uclinuxfdpiceabi targets.
> ---
>   gcc/config.gcc | 10 ++++++++--
>   1 file changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/gcc/config.gcc b/gcc/config.gcc
> index 93e2b3219b9..8c8d30ca934 100644
> --- a/gcc/config.gcc
> +++ b/gcc/config.gcc
> @@ -851,8 +851,14 @@ case ${target} in
>     tmake_file="${tmake_file} t-glibc"
>     target_has_targetcm=yes
>     target_has_targetdm=yes
> -  # Linux targets always support .init_array.
> -  gcc_cv_initfini_array=yes
> +  case $target in
> +    *-*-uclinuxfdpiceabi)
> +      ;;
> +    *)
> +      # Linux targets always support .init_array.
> +      gcc_cv_initfini_array=yes
> +      ;;
> +  esac
>     ;;
>   *-*-netbsd*)
>     tm_p_file="${tm_p_file} netbsd-protos.h"

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Do not enable DT_INIT_ARRAY/DT_FINI_ARRAY on uclinuxfdpiceabi
  2021-08-16 11:53 ` Christophe LYON
@ 2021-08-17  9:39   ` Richard Sandiford
  0 siblings, 0 replies; 3+ messages in thread
From: Richard Sandiford @ 2021-08-17  9:39 UTC (permalink / raw)
  To: Christophe LYON via Gcc-patches; +Cc: Christophe LYON, joseph

Christophe LYON via Gcc-patches <gcc-patches@gcc.gnu.org> writes:
> ping?
>
>
> On 12/08/2021 17:29, Christophe Lyon via Gcc-patches wrote:
>> Commit r12-1328 enabled DT_INIT_ARRAY/DT_FINI_ARRAY for all Linux
>> targets, but this does not work for arm-none-uclinuxfdpiceabi: it
>> makes all the execution tests fail.
>>
>> This patch restores the original behavior for uclinuxfdpiceabi.
>>
>> 2021-08-12  Christophe Lyon  <christophe.lyon@foss.st.com>
>>
>> 	gcc/
>> 	PR target/100896
>> 	* config.gcc (gcc_cv_initfini_array): Leave undefined for
>> 	uclinuxfdpiceabi targets.

OK, thanks.

Richard

>> ---
>>   gcc/config.gcc | 10 ++++++++--
>>   1 file changed, 8 insertions(+), 2 deletions(-)
>>
>> diff --git a/gcc/config.gcc b/gcc/config.gcc
>> index 93e2b3219b9..8c8d30ca934 100644
>> --- a/gcc/config.gcc
>> +++ b/gcc/config.gcc
>> @@ -851,8 +851,14 @@ case ${target} in
>>     tmake_file="${tmake_file} t-glibc"
>>     target_has_targetcm=yes
>>     target_has_targetdm=yes
>> -  # Linux targets always support .init_array.
>> -  gcc_cv_initfini_array=yes
>> +  case $target in
>> +    *-*-uclinuxfdpiceabi)
>> +      ;;
>> +    *)
>> +      # Linux targets always support .init_array.
>> +      gcc_cv_initfini_array=yes
>> +      ;;
>> +  esac
>>     ;;
>>   *-*-netbsd*)
>>     tm_p_file="${tm_p_file} netbsd-protos.h"

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-08-17  9:39 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-12 15:29 [PATCH] Do not enable DT_INIT_ARRAY/DT_FINI_ARRAY on uclinuxfdpiceabi Christophe Lyon
2021-08-16 11:53 ` Christophe LYON
2021-08-17  9:39   ` Richard Sandiford

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).