================
@@ -0,0 +1,29 @@
+// RUN: %clang_analyze_cc1 -std=c++17
-analyzer-checker=core,cplusplus.Move,alpha.cplusplus.IteratorModeling
-analyzer-config aggressive-binary-operation-simplification=true
-analyzer-config c++-container-inlining=false %s -verify -analyzer-config
display-checker-name=false
+
+#include "Inputs/system-header-simulator-cxx.h"
+
+
+//===----------------------------------------------------------------------===//
+// Test suite for test functions that require both MoveChecker.cpp and
+// IteratorModeling.cpp to be enabled.
+// NOTE: Currently the iterator dereference detection is only working when
+// IteratorModeling is enabled.
+//===----------------------------------------------------------------------===//
+
+void iteratorDerefSource() {
+ std::list<std::string> l1;
+ l1.push_back("l1");
+ std::list<std::string> l2;
+
+ std::move(l1.begin(), l1.end(), std::back_inserter(l2));
+ *l1.cbegin(); // expected-warning {{Method called on moved-from object 'l1'}}
+}
+
+void iteratorDerefDest() {
----------------
benedekaibas wrote:
> Can we demonstrate that this works on other containers too? such as vector
> for example? Can we also test associative containers, such as std::map?
>
> See my [later
> comment.](https://github.com/llvm/llvm-project/pull/196602#discussion_r3340737262)
Including `std::map` would require to extend the
`system-header-simulator-cxx.h` with not just the `inserter`, but also with the
`std::map`. I have not included that since I already cover containers and
lists, but if you think `std::maps` needed to be covered in tests then I can
make that change as well.
https://github.com/llvm/llvm-project/pull/196602
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits