[PATCH] D56160: [clang-tidy] modernize-use-trailing-return check

Bernhard Manfred Gruber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 8 02:22:19 PST 2019


bernhardmgruber marked 2 inline comments as done.
bernhardmgruber added inline comments.


================
Comment at: test/clang-tidy/modernize-use-trailing-return.cpp:1
+// RUN: %check_clang_tidy %s modernize-use-trailing-return %t -- -- --std=c++14
+
----------------
MyDeveloperDay wrote:
> nit: is there a reason here why you say C++14 when the code checks for C++11? 
Yes. The tests contain functions with deduced return types, such as `auto f();`. Those require C++14. The check itself is fine with C++11.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56160/new/

https://reviews.llvm.org/D56160





More information about the cfe-commits mailing list