This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGc8b31da1ef0a: [clang][dataflow] Allow analyzing multiple
functions in unit tests (authored by gribozavr).
Repository:
rG LLVM Github Monorepo
CHA
gribozavr2 added inline comments.
Comment at: clang/unittests/Analysis/FlowSensitive/TestingSupport.cpp:59
+if (SM.isPointWithin(Loc, BoundingRange.getBegin(),
+ BoundingRange.getEnd())) {
+ LineNumberToContent[SM.getPresumedLineNumber(Loc)] =
---
gribozavr updated this revision to Diff 491003.
gribozavr added a comment.
Address review comments
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D140859/new/
https://reviews.llvm.org/D140859
Files:
clang/unittests/Analysis/FlowSensitive/TestingSu
ymandel accepted this revision.
ymandel added inline comments.
Comment at: clang/unittests/Analysis/FlowSensitive/TestingSupport.cpp:59
+if (SM.isPointWithin(Loc, BoundingRange.getBegin(),
+ BoundingRange.getEnd())) {
+ LineNumberToContent[SM.getP
sgatev accepted this revision.
sgatev added inline comments.
This revision is now accepted and ready to land.
Comment at: clang/unittests/Analysis/FlowSensitive/TestingSupport.cpp:89-104
+ unsigned FunctionBeginOffset =
+ SourceManager.getFileOffset(Func->getBeginLoc());
+
gribozavr created this revision.
Herald added subscribers: martong, xazax.hun.
Herald added a project: All.
gribozavr requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
In unit tests for concrete dataflow analyses we typically use the
testonly