tilegx: Fix infinite loop in gen-mul-tables generator

  Hi,

Since around GCC 10, the condition `j < (INTMAX_MAX / 10)' will get
optimized into `j != 922337203685477580', which will result in an
infinite loop for certain inputs of `j'.

This patch just copies the condition already used by the -DTILEPRO
generator code, which doesn't fall into the same trap.

OK for mainline?  OK for backporting to all open release branches?

Regards,
Iain.

---
gcc/ChangeLog:

	* config/tilepro/gen-mul-tables.cc (tilegx_emit): Adjust loop
	condition to avoid overflow.
---
 gcc/config/tilepro/gen-mul-tables.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
  

On 6/22/2022 11:30 AM, Iain Buclaw via Gcc-patches wrote:
> Hi,
>
> Since around GCC 10, the condition `j < (INTMAX_MAX / 10)' will get
> optimized into `j != 922337203685477580', which will result in an
> infinite loop for certain inputs of `j'.
>
> This patch just copies the condition already used by the -DTILEPRO
> generator code, which doesn't fall into the same trap.
>
> OK for mainline?  OK for backporting to all open release branches?
Sure, but note the tile ports have been deprecated for a long time and 
I've got a TODO to finally remove them.

jeff
  

Excerpts from Jeff Law via Gcc-patches's message of Juni 22, 2022 8:16 pm:
> 
> 
> On 6/22/2022 11:30 AM, Iain Buclaw via Gcc-patches wrote:
>> Hi,
>>
>> Since around GCC 10, the condition `j < (INTMAX_MAX / 10)' will get
>> optimized into `j != 922337203685477580', which will result in an
>> infinite loop for certain inputs of `j'.
>>
>> This patch just copies the condition already used by the -DTILEPRO
>> generator code, which doesn't fall into the same trap.
>>
>> OK for mainline?  OK for backporting to all open release branches?
> Sure, but note the tile ports have been deprecated for a long time and 
> I've got a TODO to finally remove them.
> 

Noted, and I wouldn't mind seeing the back of this port either.

I encountered this as I occasionally I run a modified version of
config-list.mk to test the D front-end.  As the target isn't gone yet,
best to ensure it can still build, even when it is assumed broken.

Iain.
  

On 6/24/2022 12:19 PM, Iain Buclaw wrote:
> Excerpts from Jeff Law via Gcc-patches's message of Juni 22, 2022 8:16 pm:
>>
>> On 6/22/2022 11:30 AM, Iain Buclaw via Gcc-patches wrote:
>>> Hi,
>>>
>>> Since around GCC 10, the condition `j < (INTMAX_MAX / 10)' will get
>>> optimized into `j != 922337203685477580', which will result in an
>>> infinite loop for certain inputs of `j'.
>>>
>>> This patch just copies the condition already used by the -DTILEPRO
>>> generator code, which doesn't fall into the same trap.
>>>
>>> OK for mainline?  OK for backporting to all open release branches?
>> Sure, but note the tile ports have been deprecated for a long time and
>> I've got a TODO to finally remove them.
>>
> Noted, and I wouldn't mind seeing the back of this port either.
>
> I encountered this as I occasionally I run a modified version of
> config-list.mk to test the D front-end.  As the target isn't gone yet,
> best to ensure it can still build, even when it is assumed broken.
Understood.  I've got no objection to you going ahead and installing the 
patch if it makes your life easier...  Just wanted to make you aware if 
you were looking to do something more with that port.

jeff