Hi! On Tue, Feb 11, 2020 at 02:50:03PM +0800, Kewen.Lin wrote: > v3 changes: > - Updated _uf to _unroll for some function names.
Thanks. > By the way, should I guard the current i386/s390 loop_unroll_adjust > ealy return with (current_pass->type != RTL_PASS)? I'm inclined not > to, since this analysis isn't enabled by default, if those targets > want to adopt this analysis, they will get ICE immediately then users > can notice it and make up the gimple part check. A guard can make > ICE disappear but the hook works poorly silently, users probably miss > to update it. Just tell the maintainers of those ports about it? They are responsive, and of course they will know what they will want to do :-) Segher