For the record, these kind of "global, static" functions instead of "class"
hierarchy style of implementation is not of the same spirit of
implementation within WOPT.

Other wopt gatekeepers, please watch out for coding style deviation with
existing module. We are also here to keep the code quality of open64

Sun

On Tue, Jun 5, 2012 at 12:50 AM, <s...@open64.net> wrote:

> Author: meiye
> Date: 2012-06-04 12:50:12 -0400 (Mon, 04 Jun 2012)
> New Revision: 3937
>
> Modified:
>   trunk/osprey/be/opt/opt_proactive.cxx
>   trunk/osprey/be/opt/opt_proactive.h
> Log:
> Change return type of a function having no return value to void
>
> Modified: trunk/osprey/be/opt/opt_proactive.cxx
> ===================================================================
> --- trunk/osprey/be/opt/opt_proactive.cxx       2012-06-01 11:59:52 UTC
> (rev 3936)
> +++ trunk/osprey/be/opt/opt_proactive.cxx       2012-06-04 16:50:12 UTC
> (rev 3937)
> @@ -5294,7 +5294,7 @@
>  // }
>  //
>  // Caller of this routine should ensure the legality.
> -BOOL
> +void
>  PRO_LOOP_INTERCHANGE_TRANS::Do_swap_if(SC_NODE * sc_if)
>  {
>   SC_NODE * sc_then = sc_if->First_kid();
>
> Modified: trunk/osprey/be/opt/opt_proactive.h
> ===================================================================
> --- trunk/osprey/be/opt/opt_proactive.h 2012-06-01 11:59:52 UTC (rev 3936)
> +++ trunk/osprey/be/opt/opt_proactive.h 2012-06-04 16:50:12 UTC (rev 3937)
> @@ -337,7 +337,7 @@
>     BOOL Do_hoist_next_siblings(SC_NODE *);
>     std::pair<SC_NODE *, SC_NODE *> Do_insert_defs(SC_NODE *,
> STACK<SC_NODE *> *);
>     void Do_push_nodes(SC_NODE *, SC_NODE *, SC_NODE *, SC_NODE *);
> -    BOOL Do_swap_if(SC_NODE *);
> +    void Do_swap_if(SC_NODE *);
>     void Top_down_do_loop_unswitch(SC_NODE *);
>     void Hoist_killing_defs(SC_NODE *, SC_NODE *, int);
>     std::pair<INT, INT> Estimate_bounds(SC_NODE *, SC_NODE *);
>
>
>
> ------------------------------------------------------------------------------
> Live Security Virtual Conference
> Exclusive live event will cover all the ways today's security and
> threat landscape has changed and how IT managers can respond. Discussions
> will include endpoint security, mobile security and the latest in malware
> threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
> _______________________________________________
> Open64-devel mailing list
> Open64-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/open64-devel
>
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Open64-devel mailing list
Open64-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/open64-devel

Reply via email to