[PATCH] D130305: [clang][dataflow] Refactor ApplyBuiltinTransfer field out into DataflowAnalysisOptions struct
Stanislav Gatev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 22 07:27:39 PDT 2022
sgatev accepted this revision.
sgatev added inline comments.
================
Comment at: clang/include/clang/Analysis/FlowSensitive/DataflowAnalysis.h:66
explicit DataflowAnalysis(ASTContext &Context) : Context(Context) {}
explicit DataflowAnalysis(ASTContext &Context, bool ApplyBuiltinTransfer)
: TypeErasedDataflowAnalysis(ApplyBuiltinTransfer), Context(Context) {}
----------------
samestep wrote:
> gribozavr2 wrote:
> > Will you remove this bool overload in a later patch?
> I didn't have any plans to, but I'm fine with removing it in a later patch if people want to.
I think it makes sense to remove it. Can you already document that it's deprecated in this patch?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130305/new/
https://reviews.llvm.org/D130305
More information about the cfe-commits
mailing list