[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