[PATCH] D24716: [Polly] DeLICM/DePRE (WIP)
Tobias Grosser via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 13 07:46:46 PST 2016
grosser added a comment.
thank you for this update. I started to look again into this patch to help you with identifying the next independent change. For this I followed you advice to return always a trivial answer in makeValInst. From the original 2947 lines changed in DeLICM.cpp, I managed to get down to 2423 lines while still being able to pass reduction_embedded. You can find my changes in https://github.com/tobig/polly/commits/delicm-removal.
I am currently looking into isConflicting() and somehow believe this function can be a lot simpler after the changes I applied. However, I did not yet figure out what to change precisely. Any ideas? (Need to run now, will continue the process later on).
More information about the llvm-commits