[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