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

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 16 10:13:21 PST 2018


erichkeane added a comment.

In https://reviews.llvm.org/D54547#1301253, @arphaman wrote:

> Would it be better to deprecate the use of PTH right now for the current release, so that the users will be aware that we will remove PTH support in LLVM 9 once they upgrade to LLVM 8? Then we can remove it right after LLVM 8 branch is created.


I considered that.  However, it is:
A- Really broken
B- Only has 1 user
C- Only accessible by a cc1 option, users aren't supposed to use these anyway.

Because of this, I/we seem to have little sympathy for the users.


https://reviews.llvm.org/D54547





More information about the cfe-commits mailing list