[PATCH] D24716: [Polly] DeLICM/DePRE (WIP)

[Tobias Grosser via Phabricator via llvm-commits]

grosser added a comment.

Hi Michael,

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).

Best,
Tobias


https://reviews.llvm.org/D24716