Re: [PATCH, stage1] Make parloops gate more strict

[Bernhard Reutner-Fischer <rep dot dot dot nop at gmail dot com>]

On June 14, 2015 10:55:59 AM GMT+02:00, Tom de Vries <Tom_deVries@mentor.com> wrote:
>On 13/03/15 11:36, Richard Biener wrote:
>> On Fri, Mar 13, 2015 at 11:32 AM, Tom de Vries
><Tom_deVries@mentor.com> wrote:
>>> Hi,
>>>
>>> this patch moves a bunch of early-out tests from the parloops pass
>to the
>>> gate function.
>>>
>>> The only effect is for functions that we don't consider at all for
>>> parallelization in the parloops pass. We no longer dump those in the
>>> parloops dump file.
>>>
>>> Bootstrapped and reg-tested on x86_64.
>>>
>>> OK for stage1 trunk?
>>
>> Does it work with -fdump-passes?
>>
>
>Hi,
>
>with -fdump-passes now fixed to work on a dummy function (r222129), I'm
>
>resubmitting this patch, split up in two patches.
>
>The first patch moves two trivial early-exit tests to the parloops
>gate.
>
>The second patch moves the number_of_loops test to the parloops gate, 
>and adds a dummy loops structure in the dummy function for
>-fdump-passes.
>
>Bootstrapped and reg-tested on x86_64.
>
>Both patches OK for trunk?

diff --git a/gcc/tree-parloops.c b/gcc/tree-parloops.c
index 02f44eb..a1659a3 100644
--- a/gcc/tree-parloops.c
+++ b/gcc/tree-parloops.c
@@ -2535,10 +2535,6 @@ parallelize_loops (void)
   source_location loop_loc;
 
   /* Do not parallelize loops in the functions created by parallelization.  */
-  if (parallelized_function_p (cfun->decl))
-    return false;
-  if (cfun->has_nonlocal_label)
-    return false;
 
   gcc_obstack_init (&parloop_obstack);
   reduction_info_table_type reduction_list (10);

Now stray comment?
Stopped reading here.

Thanks,