[PATCH] D110257: [CFE][Codegen] Do not break the contiguity of static allocas.

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 23 08:59:13 PDT 2021


rnk added a comment.

In D110257#3016918 <https://reviews.llvm.org/D110257#3016918>, @hsmhsm wrote:

> That said, it is still good idea (and actually an explicitly not mandated requirement) to maintain the contiguity of the static allocas at the top of the basic block as one cluster, and it should start from FE itself.  So, this patch is still relevant.

The patch mostly affects GPU tests, so as long as the GPU folks (AMD/nvptx) are happy with the regenerated test cases, this seems fine to me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110257



More information about the cfe-commits mailing list