Hi Feng,

On Mon, Jun 01 2020, Feng Xue OS wrote:
> Clone materialization might produce some improper debug output as:
>
> Original--
>
> cloning foo/271 to foo.constprop/334
>    replace map: 0 -> xxx1->yyy
>     m_always_copy_start: 1
>     IPA adjusted parameters: foo (...)
> {
> ...
> }
>
> And a better output could be:
>
> cloning foo/271 to foo.constprop/334
>     replace map: 0 -> xxx, 1->yyy         /* separate 1 with xxx,  */
>     m_always_copy_start: 1                   /* Align with replace map */
>     IPA adjusted parameters:                /* If no adjusted parameter, 
> start a new line or omit this line */
> foo (...)
> {
> ...
> }
>
> Feng
> ---
> 2020-06-01  Feng Xue  <f...@os.amperecomputing.com>
>
> gcc/
>       * cgraphclones.c (materialize_all_clones): Adjust replace map dump.
>       * ipa-param-manipulation.c (ipa_dump_adjusted_parameters): Do not
>       dump infomation if there is no adjusted parameter.
>       * (ipa_param_adjustments::dump): Adjust prefix spaces for dump string.

This is OK, thank you.

Martin

Reply via email to