[PATCH] D43805: Optionally use nameless IR types

[Serge Pavlov via Phabricator via cfe-commits]

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.


Repository:
  rC Clang

https://reviews.llvm.org/D43805