[PATCH] D15722: [WIP][Polly] SSA Codegen

Tobias Grosser via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 30 12:23:17 PDT 2016


grosser resigned from this revision.
grosser removed a reviewer: grosser.
grosser added a comment.

I resign from this patch as it is outdated and I am also not yet really convinced that direct SSA generation is the way to go. It is a pretty complicated change which is hard to get right in all corner cases, makes the code more difficult to understand, and does not really improve things either. There is a reason clang does not generate SSA, but relies on -mem2reg ;).
I admit that there may certainly be some benefits in this, but I would suggest to start a discussion on the mailing list before anybody looks into this again.


https://reviews.llvm.org/D15722





More information about the llvm-commits mailing list