[Buildroot] [PATCH] toolchain: update gcc bug 99410

[Buildroot] [PATCH] toolchain: update gcc bug 99410

[Giulio Benetti]

Gcc bug 99410 reappeared in gcc 10.x while building belle-sip, but it's
fixed on gcc 11.x, so let's update bug conditions.

Fixes:
http://autobuild.buildroot.net/results/846597f3573d3b0d52e80627a9577d14b9348547/

Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
 toolchain/Config.in | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/toolchain/Config.in b/toolchain/Config.in
index ccdf62503d..aed8f393a3 100644
--- a/toolchain/Config.in
+++ b/toolchain/Config.in
@@ -185,11 +185,11 @@ config BR2_TOOLCHAIN_HAS_GCC_BUG_93847
 # https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99410
 # Error: branch offset out of range on Nios II. This bug
 # no longer exists in gcc 8.x but reappeared on gcc 9.x and has been
-# fixed on gcc 10.x.
+# fixed on gcc 11.x.
 config BR2_TOOLCHAIN_HAS_GCC_BUG_99140
 	bool
 	default y if !BR2_TOOLCHAIN_GCC_AT_LEAST_8
-	default y if BR2_TOOLCHAIN_GCC_AT_LEAST_9 && !BR2_TOOLCHAIN_GCC_AT_LEAST_10
+	default y if BR2_TOOLCHAIN_GCC_AT_LEAST_10 && !BR2_TOOLCHAIN_GCC_AT_LEAST_11
 	depends on BR2_nios2
 
 # https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101737
-- 
2.25.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] toolchain: update gcc bug 99410

[Thomas Petazzoni via buildroot]

On Sun,  6 Feb 2022 10:08:09 +0100
Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:

> Gcc bug 99410 reappeared in gcc 10.x while building belle-sip, but it's
> fixed on gcc 11.x, so let's update bug conditions.
> 
> Fixes:
> http://autobuild.buildroot.net/results/846597f3573d3b0d52e80627a9577d14b9348547/
> 
> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
> ---
>  toolchain/Config.in | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] toolchain: update gcc bug 99410

[Arnout Vandecappelle]

On 06/02/2022 10:08, Giulio Benetti wrote:
> Gcc bug 99410 reappeared in gcc 10.x while building belle-sip, but it's
> fixed on gcc 11.x, so let's update bug conditions.
> 
> Fixes:
> http://autobuild.buildroot.net/results/846597f3573d3b0d52e80627a9577d14b9348547/
> 
> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
> ---
>   toolchain/Config.in | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/toolchain/Config.in b/toolchain/Config.in
> index ccdf62503d..aed8f393a3 100644
> --- a/toolchain/Config.in
> +++ b/toolchain/Config.in
> @@ -185,11 +185,11 @@ config BR2_TOOLCHAIN_HAS_GCC_BUG_93847
>   # https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99410
>   # Error: branch offset out of range on Nios II. This bug
>   # no longer exists in gcc 8.x but reappeared on gcc 9.x and has been

  If this statement is true...

> -# fixed on gcc 10.x.
> +# fixed on gcc 11.x.
>   config BR2_TOOLCHAIN_HAS_GCC_BUG_99140
>   	bool
>   	default y if !BR2_TOOLCHAIN_GCC_AT_LEAST_8
> -	default y if BR2_TOOLCHAIN_GCC_AT_LEAST_9 && !BR2_TOOLCHAIN_GCC_AT_LEAST_10
> +	default y if BR2_TOOLCHAIN_GCC_AT_LEAST_10 && !BR2_TOOLCHAIN_GCC_AT_LEAST_11

  ... then this should be

	default y if BR2_TOOLCHAIN_GCC_AT_LEAST_9 && !BR2_TOOLCHAIN_GCC_AT_LEAST_11

  We don't have a nios2 GCC9 toolchain in the autobuilders so we'll never 
notice, of course.

  Regards,
  Arnout

>   	depends on BR2_nios2
>   
>   # https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101737
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] toolchain: update gcc bug 99410

[Giulio Benetti]

Hi Arnout,

On 06/02/22 12:14, Arnout Vandecappelle wrote:
> 
> 
> On 06/02/2022 10:08, Giulio Benetti wrote:
>> Gcc bug 99410 reappeared in gcc 10.x while building belle-sip, but it's
>> fixed on gcc 11.x, so let's update bug conditions.
>>
>> Fixes:
>> http://autobuild.buildroot.net/results/846597f3573d3b0d52e80627a9577d14b9348547/
>>
>> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
>> ---
>>    toolchain/Config.in | 4 ++--
>>    1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/toolchain/Config.in b/toolchain/Config.in
>> index ccdf62503d..aed8f393a3 100644
>> --- a/toolchain/Config.in
>> +++ b/toolchain/Config.in
>> @@ -185,11 +185,11 @@ config BR2_TOOLCHAIN_HAS_GCC_BUG_93847
>>    # https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99410
>>    # Error: branch offset out of range on Nios II. This bug
>>    # no longer exists in gcc 8.x but reappeared on gcc 9.x and has been
> 
>    If this statement is true...
> 
>> -# fixed on gcc 10.x.
>> +# fixed on gcc 11.x.
>>    config BR2_TOOLCHAIN_HAS_GCC_BUG_99140
>>    	bool
>>    	default y if !BR2_TOOLCHAIN_GCC_AT_LEAST_8
>> -	default y if BR2_TOOLCHAIN_GCC_AT_LEAST_9 && !BR2_TOOLCHAIN_GCC_AT_LEAST_10
>> +	default y if BR2_TOOLCHAIN_GCC_AT_LEAST_10 && !BR2_TOOLCHAIN_GCC_AT_LEAST_11
> 
>    ... then this should be
> 
> 	default y if BR2_TOOLCHAIN_GCC_AT_LEAST_9 && !BR2_TOOLCHAIN_GCC_AT_LEAST_11

but it's all negative options, so I thought I needed AT_LEAST_10 to 
cover gcc 10.x as having the bug. But it's not needed. So I only need to 
modify from !BR2_TOOLCHAIN_GCC_AT_LEAST_10 to 
!BR2_TOOLCHAIN_GCC_AT_LEAST_11 and that's it.

So I send a V2 for this.

>    We don't have a nios2 GCC9 toolchain in the autobuilders so we'll never
> notice, of course.

I've tested it locally with buildroot built toolchain.

Thank you!

Best regards
-- 
Giulio Benetti
Benetti Engineering sas

>    Regards,
>    Arnout
> 
>>    	depends on BR2_nios2
>>    
>>    # https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101737
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] toolchain: update gcc bug 99410

[Giulio Benetti]

On 06/02/22 12:33, Giulio Benetti wrote:
> Hi Arnout,
> 
> On 06/02/22 12:14, Arnout Vandecappelle wrote:
>>
>>
>> On 06/02/2022 10:08, Giulio Benetti wrote:
>>> Gcc bug 99410 reappeared in gcc 10.x while building belle-sip, but it's
>>> fixed on gcc 11.x, so let's update bug conditions.
>>>
>>> Fixes:
>>> http://autobuild.buildroot.net/results/846597f3573d3b0d52e80627a9577d14b9348547/
>>>
>>> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
>>> ---
>>>     toolchain/Config.in | 4 ++--
>>>     1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/toolchain/Config.in b/toolchain/Config.in
>>> index ccdf62503d..aed8f393a3 100644
>>> --- a/toolchain/Config.in
>>> +++ b/toolchain/Config.in
>>> @@ -185,11 +185,11 @@ config BR2_TOOLCHAIN_HAS_GCC_BUG_93847
>>>     # https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99410
>>>     # Error: branch offset out of range on Nios II. This bug
>>>     # no longer exists in gcc 8.x but reappeared on gcc 9.x and has been
>>
>>     If this statement is true...
>>
>>> -# fixed on gcc 10.x.
>>> +# fixed on gcc 11.x.
>>>     config BR2_TOOLCHAIN_HAS_GCC_BUG_99140
>>>     	bool
>>>     	default y if !BR2_TOOLCHAIN_GCC_AT_LEAST_8
>>> -	default y if BR2_TOOLCHAIN_GCC_AT_LEAST_9 && !BR2_TOOLCHAIN_GCC_AT_LEAST_10
>>> +	default y if BR2_TOOLCHAIN_GCC_AT_LEAST_10 && !BR2_TOOLCHAIN_GCC_AT_LEAST_11
>>
>>     ... then this should be
>>
>> 	default y if BR2_TOOLCHAIN_GCC_AT_LEAST_9 && !BR2_TOOLCHAIN_GCC_AT_LEAST_11
> 
> but it's all negative options, so I thought I needed AT_LEAST_10 to
> cover gcc 10.x as having the bug. But it's not needed. So I only need to
> modify from !BR2_TOOLCHAIN_GCC_AT_LEAST_10 to
> !BR2_TOOLCHAIN_GCC_AT_LEAST_11 and that's it.
> 
> So I send a V2 for this.

Not V2, I need a new one to fix it since it's already been applied.
I send it now

>>     We don't have a nios2 GCC9 toolchain in the autobuilders so we'll never
>> notice, of course.
> 
> I've tested it locally with buildroot built toolchain.
> 
> Thank you!
> 
> Best regards

-- 
Giulio Benetti
Benetti Engineering sas
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

[Buildroot] [PATCH] toolchain: fix gcc bug 99140 condition

[Giulio Benetti]

Gcc bug 99140 exhibits with gcc versions:
- up to 7.x
- 9.x
- 10.x
and doesn't show up with gcc versions:
- 8.x
- 11.x
then moving BR2_TOOLCHAIN_GCC_AT_LEAST_9 to BR2_TOOLCHAIN_GCC_AT_LEAST_10
makes gcc version 9 set as working but it's not. So let's back substitute
BR2_TOOLCHAIN_GCC_AT_LEAST_10 with BR2_TOOLCHAIN_GCC_AT_LEAST_9.

Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
 toolchain/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/toolchain/Config.in b/toolchain/Config.in
index aed8f393a3..db3f451f45 100644
--- a/toolchain/Config.in
+++ b/toolchain/Config.in
@@ -189,7 +189,7 @@ config BR2_TOOLCHAIN_HAS_GCC_BUG_93847
 config BR2_TOOLCHAIN_HAS_GCC_BUG_99140
 	bool
 	default y if !BR2_TOOLCHAIN_GCC_AT_LEAST_8
-	default y if BR2_TOOLCHAIN_GCC_AT_LEAST_10 && !BR2_TOOLCHAIN_GCC_AT_LEAST_11
+	default y if BR2_TOOLCHAIN_GCC_AT_LEAST_9 && !BR2_TOOLCHAIN_GCC_AT_LEAST_11
 	depends on BR2_nios2
 
 # https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101737
-- 
2.25.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] toolchain: fix gcc bug 99140 condition

[Arnout Vandecappelle]

On 06/02/2022 15:24, Giulio Benetti wrote:
> Gcc bug 99140 exhibits with gcc versions:
> - up to 7.x
> - 9.x
> - 10.x
> and doesn't show up with gcc versions:
> - 8.x
> - 11.x
> then moving BR2_TOOLCHAIN_GCC_AT_LEAST_9 to BR2_TOOLCHAIN_GCC_AT_LEAST_10
> makes gcc version 9 set as working but it's not. So let's back substitute
> BR2_TOOLCHAIN_GCC_AT_LEAST_10 with BR2_TOOLCHAIN_GCC_AT_LEAST_9.
> 
> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>

  Applied to master, thanks.

  Regards,
  Arnout

> ---
>   toolchain/Config.in | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/toolchain/Config.in b/toolchain/Config.in
> index aed8f393a3..db3f451f45 100644
> --- a/toolchain/Config.in
> +++ b/toolchain/Config.in
> @@ -189,7 +189,7 @@ config BR2_TOOLCHAIN_HAS_GCC_BUG_93847
>   config BR2_TOOLCHAIN_HAS_GCC_BUG_99140
>   	bool
>   	default y if !BR2_TOOLCHAIN_GCC_AT_LEAST_8
> -	default y if BR2_TOOLCHAIN_GCC_AT_LEAST_10 && !BR2_TOOLCHAIN_GCC_AT_LEAST_11
> +	default y if BR2_TOOLCHAIN_GCC_AT_LEAST_9 && !BR2_TOOLCHAIN_GCC_AT_LEAST_11
>   	depends on BR2_nios2
>   
>   # https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101737
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH] toolchain: fix gcc bug 99140 condition

[Peter Korsgaard]

>>>>> "Giulio" == Giulio Benetti <giulio.benetti@benettiengineering.com> writes:

 > Gcc bug 99140 exhibits with gcc versions:
 > - up to 7.x
 > - 9.x
 > - 10.x
 > and doesn't show up with gcc versions:
 > - 8.x
 > - 11.x
 > then moving BR2_TOOLCHAIN_GCC_AT_LEAST_9 to BR2_TOOLCHAIN_GCC_AT_LEAST_10
 > makes gcc version 9 set as working but it's not. So let's back substitute
 > BR2_TOOLCHAIN_GCC_AT_LEAST_10 with BR2_TOOLCHAIN_GCC_AT_LEAST_9.

 > Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>

Committed to 2021.11.x, thanks.

-- 
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot