[PATCH] D47405: [analyzer] Re-enable C++17-specific return value constructors.

[Artem Dergachev via Phabricator via cfe-commits]

NoQ added inline comments.


================
Comment at: test/Analysis/cxx17-mandatory-elision.cpp:185-191
+    // Check if the last destructor is an automatic destructor.
+    // A temporary destructor would have fired by now.
+#if __cplusplus >= 201703L
+    clang_analyzer_eval(v.len == 1); // expected-warning{{TRUE}}
+#else
+    clang_analyzer_eval(v.len == 3); // expected-warning{{TRUE}}
+#endif
----------------
This sounded like a good thing to check, so i added such checks in a few more places.


Repository:
  rC Clang

https://reviews.llvm.org/D47405