[PATCH] D140227: [dsymutil] Verify the keep chain when asserts are enabled

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 19 08:13:38 PST 2022


JDevlieghere marked 3 inline comments as done.
JDevlieghere added inline comments.


================
Comment at: llvm/lib/DWARFLinker/DWARFLinker.cpp:877
+
+    for (DWARFDie Child : reverse(Current.children())) {
+      Worklist.push_back(Child);
----------------
avl wrote:
> it looks not neccessary to reverse here.
You're right, but it also doesn't hurt and makes sure we visit children in order (and report them that way) which I think makes the output more like the input if there are multiple broken links for one parent.


================
Comment at: llvm/lib/DWARFLinker/DWARFLinker.cpp:886
+
+  if (!BrokenLinks.empty()) {
+    for (BrokenLink Link : BrokenLinks) {
----------------
avl wrote:
> this check looks redundant.
It guards the `report_fatal_error` below.


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

https://reviews.llvm.org/D140227



More information about the llvm-commits mailing list