[PATCH] D34002: [clang-tidy] When" -fno-exceptions is used", this warning is better to be suppressed.

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 8 15:28:00 PDT 2017


alexfh added inline comments.


================
Comment at: test/clang-tidy/misc-noexcept-move-constructor.cpp:2
+// RUN: clang-tidy %s -checks="-*,misc-noexcept-move-constructor" -- -std=c++11 \
+// RUN:   | FileCheck %s -check-prefix=CHECK-EXCEPTIONS \
+// RUN:   -implicit-check-not="{{warning|error}}:"
----------------
FlameTop wrote:
> Could I request that exceptions be explicitly enabled here (-fexceptions) as the test currently fails on targets which are configured with exceptions disabled as default? 
That would be the way to go, if we wanted to keep this change in. However, we need to revert it (which should also fix the issues you're talking about).


https://reviews.llvm.org/D34002





More information about the cfe-commits mailing list