[PATCH] D91722: [DebugInfo] Use variadic debug values to salvage BinOps and GEP instrs with non-const operands

[Hans Wennborg via Phabricator via llvm-commits]

hans added a comment.

Reverted in 386b66b2fc297cda121a3cc8a36887a6ecbcfc68 <https://reviews.llvm.org/rG386b66b2fc297cda121a3cc8a36887a6ecbcfc68> until this can be fixed.



================
Comment at: llvm/lib/Transforms/Utils/Local.cpp:1862
+  }
+  for (auto Offset : VariableOffsets) {
+    AdditionalValues.push_back(Offset.first);
----------------
aeubanks wrote:
> could this be causing non-determinism?
I tried changing VariableOffsets into a MapVector, but that wasn't enough to make the output of my reproducer deterministic.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91722