Hi Will, Thanks for the review!
on 2020/9/1 上午1:13, will schmidt wrote: > On Mon, 2020-08-31 at 14:43 +0800, Kewen.Lin via Gcc-patches wrote: >> Hi, >> >> Power9 supports vector with length in bytes load/store, this patch >> is to teach check_effective_target_vect_len_load_store to take it >> and its laters as effective vector with length targets. >> >> Also supplement the documents for has_arch_pwr*. >> >> Bootstrapped/regtested on powerpc64le-linux-gnu P8. > > Presumably also tested on P9 to see the > check_effective_target_vect_len_store check pass? :-) > Good point! I should have also mentioned that with explicit parameter vect-partial-vector-usage setting (to value 1) on P9, it's also bootstrapped/regtested. And yes, it passed, the below check depends on this change: is-effective-target: vect_partial_vectors_usage_1 1 BR, Kewen