[PATCH] D58959: [DebugInfo][ExecutionEngine] follow up for "add SectionedAddress to DebugInfo interfaces"

[Jonas Hahnfeld via Phabricator via llvm-commits]

Hahnfeld added inline comments.


================
Comment at: llvm/lib/ExecutionEngine/IntelJITEvents/IntelJITEventListener.cpp:153
+    if (auto SectOrErr = Sym.getSection())
+      SectionIndex = SectOrErr.get()->getIndex();
+
----------------
According to the documentation, `SymbolRef::getSection` returns a `section_iterator`. Do we need to check against `section_end()`? (`end_sections` in the documentation is wrong!)


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

https://reviews.llvm.org/D58959