[PATCH] D57592: Replace uses of %T with %t in from previous frontend test differential

davezarzycki added a comment. Hot fix: r352996 Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57592/new/ https://reviews.llvm.org/D57592 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cg

[PATCH] D57592: Replace uses of %T with %t in from previous frontend test differential

riccibruno added a comment. > The NetBSD buildbot breaks in these tests now: Pretty much every bot is red because of this patch (http://lab.llvm.org:8011/one_line_per_build) so yes, please fix. Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57592/new/ https://rev

[PATCH] D57592: Replace uses of %T with %t in from previous frontend test differential

krytarowski added a comment. The NetBSD buildbot breaks in these tests now: http://lab.llvm.org:8011/builders/lldb-amd64-ninja-netbsd8/builds/18721 Both tests break in a similar way: Command Output (stderr): -- + : 'RUN: at line 1' + not /home/motus/netbsd8/netbsd8/build/bin/clang -cc1

[PATCH] D57592: Replace uses of %T with %t in from previous frontend test differential

thakis added a comment. Done, thanks for the patch! Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57592/new/ https://reviews.llvm.org/D57592 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.o

[PATCH] D57592: Replace uses of %T with %t in from previous frontend test differential

This revision was automatically updated to reflect the committed changes. Closed by commit rL352971: Replace uses of %T with %t in from previous frontend test differential (authored by nico, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to co

[PATCH] D57592: Replace uses of %T with %t in from previous frontend test differential

justice_adams added a comment. @thakis thanks for the review. I don't have commit access, would you mind committing this for me? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57592/new/ https://reviews.llvm.org/D57592 _

[PATCH] D57592: Replace uses of %T with %t in from previous frontend test differential

thakis accepted this revision. thakis added a comment. This revision is now accepted and ready to land. Normally when needing a directory you'd `mkdir %t.dir` before using it, but since you don't' want the directory to exist here this should be good as-is. Thanks! Repository: rC Clang CHANG

[PATCH] D57592: Replace uses of %T with %t in from previous frontend test differential

justice_adams created this revision. justice_adams added reviewers: stella.stamenova, thakis. justice_adams added a project: clang. After committing a change I had made to a few frontend tests, it was pointed out to me that %T is being deprecated in LLVM in favor of %t. This change simply conver

[PATCH] D57592: Replace uses of %T with %t in from previous frontend test differential

justice_adams added a comment. @stella.stamenova These are the changes from our previous discussion regarding the usage of %T Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57592/new/ https://reviews.llvm.org/D57592