shafik added a subscriber: NoQ.
shafik added inline comments.

================
Comment at: clang/test/Analysis/missing-bind-temporary.cpp:10
 namespace variant_0 {
-// This variant of the code works correctly. Function foo() is not a template
-// function. Note that there are two destructors within foo().
----------------
@NoQ I had to update this test due to my fix, since this was documenting a 
flaw, I am not sure what you want to do with it.

I think it is still useful as a regression and or verification test but you may 
want to modify it differently.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151235/new/

https://reviews.llvm.org/D151235

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to