[PATCH] D51443: [dsymutil] Remove non-determinism.

Frederic Riss via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 30 13:23:08 PDT 2018


friss added inline comments.


================
Comment at: llvm/tools/dsymutil/DwarfLinker.cpp:283
+  // exists elsewhere.
+  if (!Info.Prune || !Info.Ctxt || !Info.Ctxt->getCanonicalDIEOffset())
+    return;
----------------
I think the last bit of condition is backwards. You want to set Prune to false if getCanonicalDIEOffset() returns 0.



https://reviews.llvm.org/D51443





More information about the llvm-commits mailing list