[PATCH] D150221: Add option -fkeep-static-variables to emit all static variables

Wed May 10 06:53:18 PDT 2023

erichkeane added a subscriber: aaron.ballman.
erichkeane added a comment.

In D150221#4330534 <https://reviews.llvm.org/D150221#4330534>, @hubert.reinterpretcast wrote:

> In D150221#4330504 <https://reviews.llvm.org/D150221#4330504>, @MaskRay wrote:
>
>> Can you give a more compelling reason that this option is needed?
>
> This is intended to prevent "excessive transformation" to enable migration of existing applications (using a non-Clang compiler) where users further manipulate the object or assembly after compilation.

I don't get what you mean by this?  I don't currently see motivation for this?  @aaron.ballman would have to make a decision here eventually, but I don't think he'd be compelled by this either.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150221