[PATCH] D144476: [GlobalOpt] Extend logic in SRA heuristic to skip stores of initializer.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 7 05:12:24 PST 2023


fhahn added a comment.

In D144476#4171586 <https://reviews.llvm.org/D144476#4171586>, @nikic wrote:

> LGTM, though I think it might be cleaner if we collected the initializer part during the construction of `GlobalPart`, rather than calculating it twice, here and during the SRA split.

Posted D145490 <https://reviews.llvm.org/D145490> as follow-up


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144476



More information about the llvm-commits mailing list