[PATCH] D59555: [analyzer] Add yaml parser to GenericTaintChecker

[Alex Langford via Phabricator via llvm-commits]

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