[PATCH] D35396: [lit] Remove %T

Peter Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 14 12:52:40 PDT 2018


Lekensteyn added a comment.

In https://reviews.llvm.org/D35396#1132500, @kubamracek wrote:

> In https://reviews.llvm.org/D35396#1132499, @Lekensteyn wrote:
>
> > Hi, what is the status with this patch? It was referred to by https://reviews.llvm.org/D36495 (which is committed), and there still seem to be a lot of users of `%T`.
>
>
> Yes, that's why this one didn't land yet. We need to get rid of all the %T users first.


Perhaps you could already adjust the documentation and mark `%T` as deprecated. I have honestly forgotten about this patch and used `%T` again in some new tests because it was documented (without any caveats).


https://reviews.llvm.org/D35396





More information about the llvm-commits mailing list