Re: [PATCH] Fix hash_operand for fields of a CONSTRUCTOR.

2019-11-06 Thread Jeff Law
On 11/5/19 1:35 AM, Martin Liška wrote: > On 11/4/19 4:24 PM, Jeff Law wrote: >> On 11/4/19 6:36 AM, Richard Biener wrote: >>> On Mon, Nov 4, 2019 at 2:35 PM Richard Biener >>>  wrote: On Mon, Nov 4, 2019 at 10:09 AM Martin Liška  wrote: > > On 11/1/19 10:51 PM, Jeff Law wrote: >>

Re: [PATCH] Fix hash_operand for fields of a CONSTRUCTOR.

2019-11-05 Thread Martin Liška
On 11/4/19 4:24 PM, Jeff Law wrote: On 11/4/19 6:36 AM, Richard Biener wrote: On Mon, Nov 4, 2019 at 2:35 PM Richard Biener wrote: On Mon, Nov 4, 2019 at 10:09 AM Martin Liška wrote: On 11/1/19 10:51 PM, Jeff Law wrote: On 10/31/19 10:01 AM, Martin Liška wrote: Hi. operand_equal_p can p

Re: [PATCH] Fix hash_operand for fields of a CONSTRUCTOR.

2019-11-04 Thread Jeff Law
On 11/4/19 6:36 AM, Richard Biener wrote: > On Mon, Nov 4, 2019 at 2:35 PM Richard Biener > wrote: >> >> On Mon, Nov 4, 2019 at 10:09 AM Martin Liška wrote: >>> >>> On 11/1/19 10:51 PM, Jeff Law wrote: On 10/31/19 10:01 AM, Martin Liška wrote: > Hi. > > operand_equal_p can proper

Re: [PATCH] Fix hash_operand for fields of a CONSTRUCTOR.

2019-11-04 Thread Jeff Law
On 11/4/19 6:35 AM, Richard Biener wrote: > On Mon, Nov 4, 2019 at 10:09 AM Martin Liška wrote: >> >> On 11/1/19 10:51 PM, Jeff Law wrote: >>> On 10/31/19 10:01 AM, Martin Liška wrote: Hi. operand_equal_p can properly handle situation where we have a CONSTRUCTOR where indices a

Re: [PATCH] Fix hash_operand for fields of a CONSTRUCTOR.

2019-11-04 Thread Richard Biener
On Mon, Nov 4, 2019 at 2:35 PM Richard Biener wrote: > > On Mon, Nov 4, 2019 at 10:09 AM Martin Liška wrote: > > > > On 11/1/19 10:51 PM, Jeff Law wrote: > > > On 10/31/19 10:01 AM, Martin Liška wrote: > > >> Hi. > > >> > > >> operand_equal_p can properly handle situation where we have a CONSTRUC

Re: [PATCH] Fix hash_operand for fields of a CONSTRUCTOR.

2019-11-04 Thread Richard Biener
On Mon, Nov 4, 2019 at 10:09 AM Martin Liška wrote: > > On 11/1/19 10:51 PM, Jeff Law wrote: > > On 10/31/19 10:01 AM, Martin Liška wrote: > >> Hi. > >> > >> operand_equal_p can properly handle situation where we have a CONSTRUCTOR > >> where indices are NULL: > >> > >>if (!operand_equ

Re: [PATCH] Fix hash_operand for fields of a CONSTRUCTOR.

2019-11-04 Thread Martin Liška
On 11/1/19 10:51 PM, Jeff Law wrote: On 10/31/19 10:01 AM, Martin Liška wrote: Hi. operand_equal_p can properly handle situation where we have a CONSTRUCTOR where indices are NULL: if (!operand_equal_p (c0->value, c1->value, flags) /* In GIMPLE the indexes can b

Re: [PATCH] Fix hash_operand for fields of a CONSTRUCTOR.

2019-11-01 Thread Jeff Law
On 10/31/19 10:01 AM, Martin Liška wrote: > Hi. > > operand_equal_p can properly handle situation where we have a CONSTRUCTOR > where indices are NULL: > > if (!operand_equal_p (c0->value, c1->value, flags) > /* In GIMPLE the indexes can be either NULL or matching i. >

[PATCH] Fix hash_operand for fields of a CONSTRUCTOR.

2019-10-31 Thread Martin Liška
Hi. operand_equal_p can properly handle situation where we have a CONSTRUCTOR where indices are NULL: if (!operand_equal_p (c0->value, c1->value, flags) /* In GIMPLE the indexes can be either NULL or matching i. Double check this so we won't ge