[PATCH] D44912: [clang-doc] Removing -Wunused-variable warning

2018-03-26 Thread Eugene Zelenko via Phabricator via cfe-commits
Eugene.Zelenko added a comment. I think it's fine to commit trivial fixes without going via review. Repository: rL LLVM https://reviews.llvm.org/D44912 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/

[PATCH] D44912: [clang-doc] Removing -Wunused-variable warning

2018-03-26 Thread Julie Hockett via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL328588: [clang-doc] Removing -Wunused-variable warning (authored by juliehockett, committed by ). Herald added subscribers: llvm-commits, klimek. Changed prior to commit: https://reviews.llvm.org/D44912

[PATCH] D44912: [clang-doc] Removing -Wunused-variable warning

2018-03-26 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri accepted this revision. lebedev.ri added a comment. LGTM. https://reviews.llvm.org/D44912 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D44912: [clang-doc] Removing -Wunused-variable warning

2018-03-26 Thread Jake Ehrlich via Phabricator via cfe-commits
jakehehrlich accepted this revision. jakehehrlich added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D44912 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/li

[PATCH] D44912: [clang-doc] Removing -Wunused-variable warning

2018-03-26 Thread Julie Hockett via Phabricator via cfe-commits
juliehockett created this revision. juliehockett added reviewers: jakehehrlich, lebedev.ri, sammccall. juliehockett added a project: clang-tools-extra. Herald added subscribers: JDevlieghere, aprantl. Warning was appearing in release with debug info build, this removes it. https://reviews.llvm.o