[PATCH] D43805: Optionally use nameless IR types

Serge Pavlov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 6 21:18:49 PST 2018

sepavloff added a comment.

In https://reviews.llvm.org/D43805#1029479, @pcc wrote:

> > If the backend will be changed so that it will not depend on IR type names
> Are you referring to https://reviews.llvm.org/D43199? If so it seems to me that this should be a cc1 flag that defaults to whether `-flto=thin` is passed. In any case it seems like a bad idea to deliberately generate different code depending on whether we were compiled with NDEBUG.

No, I try to implement alternative approach, to solve the problem targeted in https://reviews.llvm.org/D40508. If  IR type names are only for human readability, than using them in opaque type resolution does not look reasonable. Probably, more correct way is to make type merge only as a side effect of merge of other entities, which may have "real" names, that is functions and variables.

  rC Clang


More information about the cfe-commits mailing list