[PATCH] D98589: [llvm] [dwarf] Fix DWARFListTableHeader::getOffsetEntry off-by-one

[Pavel Labath via Phabricator via llvm-commits]

labath accepted this revision.
labath added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/unittests/DebugInfo/DWARF/DWARFListTableTest.cpp:95-96
+  Optional<uint64_t> Offset0 = Header.getOffsetEntry(Extractor, 0);
+  EXPECT_TRUE(!!Offset0);
+  EXPECT_EQ(*Offset0, uint64_t(4));
+  Optional<uint64_t> Offset1 = Header.getOffsetEntry(Extractor, 1);
----------------
I think `EXPECT_EQ(Offset0, uint64_t(4));` (note missing `*`) should be enough.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D98589/new/

https://reviews.llvm.org/D98589