On May 23, 2018 5:29:46 PM GMT+02:00, Richard Sandiford <richard.sandif...@arm.com> wrote: >Richard Biener <rguent...@suse.de> writes: >> This patch splits GROUP_* into DR_GROUP_* and REDUC_GROUP_* as a step >> towards sharing dataref and dependence analysis between vectorization >> analysis of different vector sizes (and then comparing costs and >choosing >> the best one). >> >> The next patch will move the DR_GROUP_* fields to the DR_AUX >vectorizer >> data structure which means that the DR group analysis could >eventually >> be used from outside of the vectorizer. >> >> Bootstrap / regtest running on x86_64-unknown-linux-gnu. >> >> I'm considering applying this independently because of the churn it >> generates to make my life easier. >> >> Richard - I remember you talking about patches walking in the very >> same direction so please speak up if this disrupts your work and >> lets coordinate. > >I don't think this patch clashes with what I was trying, so please >go ahead. I was looking at patches to remove the globalness of >vinfo_for_stmt, mostly by avoiding it wherever possible and using >stmt_info instead of gimple *. That's quite a lot of churn but leaves >very few vinfo_for_stmts left.
Ah, that's indeed nice. >It also included an attempt to clean up how we handle getting the >vector versions of operands. Ambitious ;) I've tried this many times... Richard. >Thanks, >Richard
