[PATCH] D153491: [dataflow] Avoid copying environment

Mon Jun 26 00:49:49 PDT 2023

xazax.hun added a comment.

In D153491#4445051 <https://reviews.llvm.org/D153491#4445051>, @ymandel wrote:

> In D153491#4443704 <https://reviews.llvm.org/D153491#4443704>, @xazax.hun wrote:
>
>> This sounds extremely error-prone to me. In case copying the analysis state has side effects like this, I would argue we want such operations to be really explicit. What do you think?
>
> Can you expand on this concern? Are you referring to the removal of the unintended copy (ie. this patch), or raising a concern about the how the underlying system handles copies altogether?

The latter, https://reviews.llvm.org/D153674 should address my concerns.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153491