[PATCH] D41296: Limit size of SROA - generated register names

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 21 09:26:33 PST 2017

mehdi_amini added a comment.

In https://reviews.llvm.org/D41296#962279, @serge-sans-paille wrote:

> In https://reviews.llvm.org/D41296#962215, @hfinkel wrote:
> > Why don't we just cap all names at some point (and then just start adding numbers as we generally do to break degeneracies). It seems like, otherwise, we'll end up with these kinds of fixes in many places. Fixing this in one common place seems better.
> I thought of this, very happy you suggest we go that way. In a release build, we could activate that by default and keep the names in debug build.

Isn't it *already* the case?
As I understand, the suggestion is to cap the length of the names in Debug build.



More information about the llvm-commits mailing list