[PATCH] D137497: [ArgumentPromotion] Allow the frontend to specify the maximum number of elements to promote on a per-function basis via metadata.

Patrick Walton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 7 11:30:42 PST 2022


pcwalton added a comment.

In D137497#3912021 <https://reviews.llvm.org/D137497#3912021>, @lebedev.ri wrote:

> FWIW, that is rather exactly what i'm doing in https://reviews.llvm.org/D113520. It seems to work, but is needs some legality check, because it can end up trying to promote backing alloca, over and over again.

Yeah, I fixed your patch, but I think it still miscompiles part of rustc. I'm working on a fix now. :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137497



More information about the llvm-commits mailing list