[PATCH] D54547: PTH-- Remove feature entirely-

Bruno Ricci via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 15 07:59:30 PST 2018


riccibruno added a comment.

In https://reviews.llvm.org/D54547#1299883, @erichkeane wrote:

> Added to the release notes.  Also, an email was sent out to cfe-dev.
>
> @riccibruno and I are separately looking into IdentifierInfo, because it seems that there are some pretty massive optimization opportunities if we can remove PTH/indirect identifiers.


As discussed on IRC:
In any case this is not blocking the removal of PTH.  The code around `IdentifierInfo`
is very performance sensitive and it will take some time to evaluate the possible
changes, if any.


https://reviews.llvm.org/D54547





More information about the cfe-commits mailing list