On Mon, May 27, 2019 at 11:33:00PM +0200, Thomas Schwinge wrote:
> Why exactly are you objecting that the compiler produces useful
> intermediate dumps, understandable my machine and human?
While still not convinced we need this, if we add it, you are still
printing a lot of stuff that is useless to humans and makes it harder to
read, plus you unnecessarily print it in two spots rather than just one.
Printing Mapping on each line just adds clutter, just print the target
stmt with TDF_LINENO followed by "Mappings:\n"
once with the location string, replace the break; at the end of
OMP_CLAUSE_FIRSTPRIVATE: handling with nc = c; tkind_zero = tkind; goto
label; which will be right before if (nc && nc != c) and do the printing
there. print_generic_expr the clause (c), the name of function seems
completely useless to me (and trying to match it in scan-tree-dump when it
can change any time doesn't make sense), TREE_PURPOSE doesn't seem to be
much useful either, print s expression (size), if tkind == tkind_zero,
just print one number (tkind & ((HOST_WIDE_INT_1U << talign_shift) - 1),
otherwise print two numbers extracted similar way from also tkind_zero,
finally the alignment.
Jakub
