================
@@ -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

Reply via email to