[PATCH] D125776: [llvm-dva] 01 - Interval tree
Paul Robinson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 18 06:11:46 PDT 2022
probinson added inline comments.
================
Comment at: llvm/unittests/ADT/IntervalTreeTest.cpp:102
+ Intervals = Tree.getContaining(Point);
+ ASSERT_TRUE(Intervals.size() == 1);
+ checkData(Point, Intervals[0], 10, 20, 10.20);
----------------
Better to use `ASSERT_EQ(Intervals.size(), 1)` here and below. Googletest will provide a better diagnostic on failure, this way.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125776/new/
https://reviews.llvm.org/D125776
More information about the llvm-commits
mailing list