xiaobai added inline comments.

================
Comment at: cfe/trunk/lib/StaticAnalyzer/Checkers/GenericTaintChecker.cpp:74
+    TaintConfiguration() = default;
+    TaintConfiguration(const TaintConfiguration &) = delete;
+    TaintConfiguration(TaintConfiguration &&) = default;
----------------
Is there a particular reason this is marked as delete? This present an issue 
because the Optional constructed in `ento::registerGenericTaintChecker` will 
use this constructor. I don't hit this issue with any recent version of clang 
(tried with 6, 7, and 8) but I am hitting it with clang 3.8 on the swift-ci 
ubuntu bots.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59555



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

Reply via email to