This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG64f7b2d4bf92: [clang][dataflow] Change `transfer` function
to update lattice element in place. (authored by ymandel).
Repository:
rG LLVM Github
ymandel updated this revision to Diff 398612.
ymandel added a comment.
address comments
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116834/new/
https://reviews.llvm.org/D116834
Files:
sgatev accepted this revision.
sgatev added inline comments.
This revision is now accepted and ready to land.
Comment at:
clang/unittests/Analysis/FlowSensitive/TypeErasedDataflowAnalysisTest.cpp:167
- FunctionCallLattice transfer(const Stmt *S, const FunctionCallLattice ,
-
ymandel created this revision.
ymandel added reviewers: sgatev, gribozavr2.
ymandel requested review of this revision.
Herald added a project: clang.
Currently, the transfer function returns a new lattice element, which forces an
unnecessary copy on processing each CFG statement.
Repository: