Re: [PATCH] convert MIN_EXPR operands to the same type (PR 87059)

2018-08-28 Thread Jeff Law
On 08/27/2018 02:32 AM, Richard Biener wrote: > On Sat, Aug 25, 2018 at 9:14 PM Jeff Law wrote: >> >> On 08/24/2018 01:06 PM, Martin Sebor wrote: >>> PR 87059 points out an ICE in the recently enhanced VRP code >>> that was traced back to a MIN_EXPR built out of operands of >>> types with differen

Re: [PATCH] convert MIN_EXPR operands to the same type (PR 87059)

2018-08-27 Thread Richard Biener
On Mon, Aug 27, 2018 at 5:51 PM Martin Sebor wrote: > > On 08/27/2018 09:28 AM, Richard Biener wrote: > > On Mon, Aug 27, 2018 at 4:41 PM Martin Sebor wrote: > >> > >> On 08/27/2018 02:32 AM, Richard Biener wrote: > >>> On Sat, Aug 25, 2018 at 9:14 PM Jeff Law wrote: > > On 08/24/2018

Re: [PATCH] convert MIN_EXPR operands to the same type (PR 87059)

2018-08-27 Thread Martin Sebor
On 08/27/2018 09:28 AM, Richard Biener wrote: On Mon, Aug 27, 2018 at 4:41 PM Martin Sebor wrote: On 08/27/2018 02:32 AM, Richard Biener wrote: On Sat, Aug 25, 2018 at 9:14 PM Jeff Law wrote: On 08/24/2018 01:06 PM, Martin Sebor wrote: PR 87059 points out an ICE in the recently enhanced V

Re: [PATCH] convert MIN_EXPR operands to the same type (PR 87059)

2018-08-27 Thread Richard Biener
On Mon, Aug 27, 2018 at 5:27 PM Jeff Law wrote: > > On 08/27/2018 02:32 AM, Richard Biener wrote: > > On Sat, Aug 25, 2018 at 9:14 PM Jeff Law wrote: > >> > >> On 08/24/2018 01:06 PM, Martin Sebor wrote: > >>> PR 87059 points out an ICE in the recently enhanced VRP code > >>> that was traced back

Re: [PATCH] convert MIN_EXPR operands to the same type (PR 87059)

2018-08-27 Thread Richard Biener
On Mon, Aug 27, 2018 at 4:41 PM Martin Sebor wrote: > > On 08/27/2018 02:32 AM, Richard Biener wrote: > > On Sat, Aug 25, 2018 at 9:14 PM Jeff Law wrote: > >> > >> On 08/24/2018 01:06 PM, Martin Sebor wrote: > >>> PR 87059 points out an ICE in the recently enhanced VRP code > >>> that was traced

Re: [PATCH] convert MIN_EXPR operands to the same type (PR 87059)

2018-08-27 Thread Jeff Law
On 08/27/2018 02:32 AM, Richard Biener wrote: > On Sat, Aug 25, 2018 at 9:14 PM Jeff Law wrote: >> >> On 08/24/2018 01:06 PM, Martin Sebor wrote: >>> PR 87059 points out an ICE in the recently enhanced VRP code >>> that was traced back to a MIN_EXPR built out of operands of >>> types with differen

Re: [PATCH] convert MIN_EXPR operands to the same type (PR 87059)

2018-08-27 Thread Martin Sebor
On 08/27/2018 02:32 AM, Richard Biener wrote: On Sat, Aug 25, 2018 at 9:14 PM Jeff Law wrote: On 08/24/2018 01:06 PM, Martin Sebor wrote: PR 87059 points out an ICE in the recently enhanced VRP code that was traced back to a MIN_EXPR built out of operands of types with different sign by expan

Re: [PATCH] convert MIN_EXPR operands to the same type (PR 87059)

2018-08-27 Thread Martin Liška
On 08/27/2018 10:32 AM, Richard Biener wrote: > On Sat, Aug 25, 2018 at 9:14 PM Jeff Law wrote: >> >> On 08/24/2018 01:06 PM, Martin Sebor wrote: >>> PR 87059 points out an ICE in the recently enhanced VRP code >>> that was traced back to a MIN_EXPR built out of operands of >>> types with differen

Re: [PATCH] convert MIN_EXPR operands to the same type (PR 87059)

2018-08-27 Thread Richard Biener
On Sat, Aug 25, 2018 at 9:14 PM Jeff Law wrote: > > On 08/24/2018 01:06 PM, Martin Sebor wrote: > > PR 87059 points out an ICE in the recently enhanced VRP code > > that was traced back to a MIN_EXPR built out of operands of > > types with different sign by expand_builtin_strncmp(). > > > > The at

Re: [PATCH] convert MIN_EXPR operands to the same type (PR 87059)

2018-08-26 Thread H.J. Lu
On Sat, Aug 25, 2018 at 12:13 PM, Jeff Law wrote: > On 08/24/2018 01:06 PM, Martin Sebor wrote: >> PR 87059 points out an ICE in the recently enhanced VRP code >> that was traced back to a MIN_EXPR built out of operands of >> types with different sign by expand_builtin_strncmp(). >> >> The attache

Re: [PATCH] convert MIN_EXPR operands to the same type (PR 87059)

2018-08-25 Thread Jeff Law
On 08/24/2018 01:06 PM, Martin Sebor wrote: > PR 87059 points out an ICE in the recently enhanced VRP code > that was traced back to a MIN_EXPR built out of operands of > types with different sign by expand_builtin_strncmp(). > > The attached patch adjusts the function to make sure both > operands

[PATCH] convert MIN_EXPR operands to the same type (PR 87059)

2018-08-24 Thread Martin Sebor
PR 87059 points out an ICE in the recently enhanced VRP code that was traced back to a MIN_EXPR built out of operands of types with different sign by expand_builtin_strncmp(). The attached patch adjusts the function to make sure both operands have the same type, and to make these mismatches easie