Re: [PATCH] Use more DECL_BUILT_IN_P macro.

2018-08-28 Thread H.J. Lu
> > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. > > Ready to be installed? This caused: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87130 H.J.

Re: [PATCH] Use more DECL_BUILT_IN_P macro.

2018-08-28 Thread Martin Liška
On 08/27/2018 06:02 PM, Martin Sebor wrote: > On 08/27/2018 01:20 AM, Martin Liška wrote: >> +/* For a FUNCTION_DECL NODE, nonzero means a built in function of a >> + standard library or more generally a built in function that is >> + recognized by optimizers and expanders. Hi. Thanks for help wi

Re: [PATCH] Use more DECL_BUILT_IN_P macro.

2018-08-27 Thread Martin Sebor
On 08/27/2018 01:20 AM, Martin Liška wrote: +/* For a FUNCTION_DECL NODE, nonzero means a built in function of a + standard library or more generally a built in function that is + recognized by optimizers and expanders. I'm a little confused by this description: the last part makes me wonder ab

Re: [PATCH] Use more DECL_BUILT_IN_P macro.

2018-08-27 Thread Richard Biener
On Mon, Aug 27, 2018 at 9:20 AM Martin Liška wrote: > > On 08/23/2018 03:58 PM, Richard Biener wrote: > > On Thu, Aug 23, 2018 at 3:30 PM Martin Liška wrote: > >> > >> On 08/23/2018 01:58 PM, Richard Biener wrote: > >>> On Thu, Aug 23, 2018 at 12:46 PM Martin Liška wrote: > > On 08/20/

Re: [PATCH] Use more DECL_BUILT_IN_P macro.

2018-08-27 Thread Martin Liška
On 08/23/2018 03:58 PM, Richard Biener wrote: > On Thu, Aug 23, 2018 at 3:30 PM Martin Liška wrote: >> >> On 08/23/2018 01:58 PM, Richard Biener wrote: >>> On Thu, Aug 23, 2018 at 12:46 PM Martin Liška wrote: On 08/20/2018 10:34 AM, Richard Biener wrote: > On Wed, Aug 15, 2018 at 2:

Re: [PATCH] Use more DECL_BUILT_IN_P macro.

2018-08-23 Thread Martin Liška
On 08/23/2018 03:58 PM, Richard Biener wrote: > On Thu, Aug 23, 2018 at 3:30 PM Martin Liška wrote: >> >> On 08/23/2018 01:58 PM, Richard Biener wrote: >>> On Thu, Aug 23, 2018 at 12:46 PM Martin Liška wrote: On 08/20/2018 10:34 AM, Richard Biener wrote: > On Wed, Aug 15, 2018 at 2:

Re: [PATCH] Use more DECL_BUILT_IN_P macro.

2018-08-23 Thread Richard Biener
On Thu, Aug 23, 2018 at 3:30 PM Martin Liška wrote: > > On 08/23/2018 01:58 PM, Richard Biener wrote: > > On Thu, Aug 23, 2018 at 12:46 PM Martin Liška wrote: > >> > >> On 08/20/2018 10:34 AM, Richard Biener wrote: > >>> On Wed, Aug 15, 2018 at 2:52 PM Martin Liška wrote: > > On 08/14/

Re: [PATCH] Use more DECL_BUILT_IN_P macro.

2018-08-23 Thread Martin Liška
On 08/23/2018 01:58 PM, Richard Biener wrote: > On Thu, Aug 23, 2018 at 12:46 PM Martin Liška wrote: >> >> On 08/20/2018 10:34 AM, Richard Biener wrote: >>> On Wed, Aug 15, 2018 at 2:52 PM Martin Liška wrote: On 08/14/2018 06:02 PM, Martin Sebor wrote: > On 08/14/2018 03:06 AM, Mart

Re: [PATCH] Use more DECL_BUILT_IN_P macro.

2018-08-23 Thread Richard Biener
On Thu, Aug 23, 2018 at 12:46 PM Martin Liška wrote: > > On 08/20/2018 10:34 AM, Richard Biener wrote: > > On Wed, Aug 15, 2018 at 2:52 PM Martin Liška wrote: > >> > >> On 08/14/2018 06:02 PM, Martin Sebor wrote: > >>> On 08/14/2018 03:06 AM, Martin Liška wrote: > Hi. > > The patch

Re: [PATCH] Use more DECL_BUILT_IN_P macro.

2018-08-23 Thread Martin Liška
On 08/20/2018 10:34 AM, Richard Biener wrote: > On Wed, Aug 15, 2018 at 2:52 PM Martin Liška wrote: >> >> On 08/14/2018 06:02 PM, Martin Sebor wrote: >>> On 08/14/2018 03:06 AM, Martin Liška wrote: Hi. The patch adds more usages of the new macro. I hope it improves readability

Re: [PATCH] Use more DECL_BUILT_IN_P macro.

2018-08-20 Thread Richard Biener
On Wed, Aug 15, 2018 at 2:52 PM Martin Liška wrote: > > On 08/14/2018 06:02 PM, Martin Sebor wrote: > > On 08/14/2018 03:06 AM, Martin Liška wrote: > >> Hi. > >> > >> The patch adds more usages of the new macro. I hope it improves > >> readability of code. > > > > I think it does :) I see that mo

Re: [PATCH] Use more DECL_BUILT_IN_P macro.

2018-08-15 Thread Martin Liška
On 08/14/2018 06:02 PM, Martin Sebor wrote: > On 08/14/2018 03:06 AM, Martin Liška wrote: >> Hi. >> >> The patch adds more usages of the new macro. I hope it improves >> readability of code. > > I think it does :)  I see that most invocations of it in your > patch are with BUILT_IN_NORMAL as the s

Re: [PATCH] Use more DECL_BUILT_IN_P macro.

2018-08-14 Thread Martin Sebor
On 08/14/2018 03:06 AM, Martin Liška wrote: Hi. The patch adds more usages of the new macro. I hope it improves readability of code. I think it does :) I see that most invocations of it in your patch are with BUILT_IN_NORMAL as the second argument. Is the argument implied by the last argumen