hokein wrote:

> Can you also add a lit test for:
> 
> ```c++
> std::vector<std::string> getVector();
> std::set<std::string> s;
> s.insert(getVector().begin(), getVector().end()); // FIXME: taking iterator 
> of a temporary container without immediate dereference is almost always wrong.
> ```

I think this is a different issue, I'd prefer not adding it to this test file.

https://github.com/llvm/llvm-project/pull/118013
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to