[PATCH] D115235: [clang][dataflow] Implement a basic algorithm for dataflow analysis

[Stanislav Gatev via Phabricator via cfe-commits]

sgatev marked 4 inline comments as done.
sgatev added inline comments.


================
Comment at: clang/lib/Analysis/FlowSensitive/TypeErasedDataflowAnalysis.cpp:31
+/// states of its predecessor basic blocks.
+TypeErasedDataflowAnalysisState computeBlockInputState(
+    std::vector<llvm::Optional<TypeErasedDataflowAnalysisState>> &BlockStates,
----------------
ymandel wrote:
> should this be declared in the header? If not, please mark static.
Marked it as static.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115235