[PATCH] D131809: [clang][dataflow] Add an option for context-sensitive depth

Sam Estep via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 15 12:54:39 PDT 2022


samestep added inline comments.


================
Comment at: clang/lib/Analysis/FlowSensitive/Transfer.cpp:665
+    if (!(Options.ContextSensitiveOpts &&
+          Env.canDescend(Options.ContextSensitiveOpts->Depth, F)))
       return;
----------------
samestep wrote:
> xazax.hun wrote:
> > Alternatively, `canDescend` could get the optional `ContextSensitiveOpts` and we can do all the checking there. 
> Ah, good idea!
Oh... actually that doesn't work quite so nicely, because it introduces a circular dependency between `Transfer.h` and `DataflowEnvironment.h`. I'll leave it as-is for now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131809



More information about the cfe-commits mailing list