[PATCH] D20560: [new PM] port LoopAccessAnalysis to new pass manager (part-1)
Chandler Carruth via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 6 17:29:54 PDT 2016
chandlerc added a comment.
The number of different changes here make it really hard to review.
Can you instead actually send out the individual changes?
I also don't see any benefit to renaming the file. I think the "Info" suffix, as common as it is in LLVM, is probably a mistake. I'm not sure its worth going and fixing everything to use a different suffix, but I like the current file name better than the one you propose here. (Just my opinion, if others have a different view from working more closely with the loop access stuff or vectorization stuff, totally fine with that...)
http://reviews.llvm.org/D20560
More information about the llvm-commits
mailing list