[PATCH] D95329: [llvm-link] Fix crash when materializing appending global

[Artem Belevich via Phabricator via llvm-commits]

tra added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/ValueMapper.cpp:822-824
+      // mapAppendingVariable call can change AppendingInits if initalizer for
+      // the variable depends on another appending global, because of that inits
+      // need to be extracted and updated before the call.
----------------
Does the crash happen because `mapAppendingVariable` attempts to iterate over an array that may change by the loop?
If that's the case, the fix (making a temp copy with relevant items) should probably be done there.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95329