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

Wed Nov 10 13:29:52 PST 2021

glandium added a comment.

In D91722#3121293 <https://reviews.llvm.org/D91722#3121293>, @StephenTozer wrote:

> In D91722#3101440 <https://reviews.llvm.org/D91722#3101440>, @glandium wrote:
>
>> The relanding in 47633af9d4a8b93f50cb711cf23489736e0226f1 <https://reviews.llvm.org/rG47633af9d4a8b93f50cb711cf23489736e0226f1> broke determinism for Firefox.
>
> Has the non-determinism problem been fixed with the landing of rG3c47c5ca13b8a502de3272e8105548715947b7a8 <https://reviews.llvm.org/rG3c47c5ca13b8a502de3272e8105548715947b7a8>?

That seems to fix the non-determinism issue introduced in D106585 <https://reviews.llvm.org/D106585>.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91722