[PATCH] D82139: [DWARFYAML][debug_info] Fix array index out of bounds error

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 19 09:45:05 PDT 2020


MaskRay accepted this revision.
MaskRay added a comment.

Looks great!



================
Comment at: llvm/lib/ObjectYAML/DWARFVisitor.cpp:49
     onStartCompileUnit(Unit);
+    if (Unit.Entries.size() == 0)
+      continue;
----------------
Use `empty()`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82139





More information about the llvm-commits mailing list