[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