rengolin added a comment.
Great work, Tim! Thank you!
http://reviews.llvm.org/D17999
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
On 9 March 2016 at 11:12, Sanjay Patel wrote:
> Nothing to do with the patch itself (although it looks great and thanks!) -
> is the CHECK line generator script available somewhere? I'd love to be able
> to use that.
It's horribly hacky and fragile, but what I've got should be attached
here. Pa
spatel added a subscriber: spatel.
spatel added a comment.
Nothing to do with the patch itself (although it looks great and thanks!) - is
the CHECK line generator script available somewhere? I'd love to be able to use
that.
http://reviews.llvm.org/D17999
t.p.northover closed this revision.
t.p.northover added a comment.
Thanks Eric. Committed as r263048.
http://reviews.llvm.org/D17999
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
echristo added a subscriber: echristo.
echristo accepted this revision.
echristo added a reviewer: echristo.
echristo added a comment.
This revision is now accepted and ready to land.
This is a compromise I can live with :)
Thanks for doing this Tim.
-eric
http://reviews.llvm.org/D17999
___
