[PATCH] D124514: [ArgPromotion] Make a non-byval promotion attempt first

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 2 09:27:54 PDT 2022


aeubanks added a comment.

In D124514#3476973 <https://reviews.llvm.org/D124514#3476973>, @psamolysov wrote:

> Change the test to use the legacy pass manager, unfortunately the buildbot doesn't accept tests with arguments for the `opt` tool in the following format: `-passes=function(sroa),cgscc(argpromotion)`.

this should work if you add quotes, like `-passes='function(sroa),cgscc(argpromotion)'`

but is there a reason we're not using the output of `-passes=sroa` as the input IR and only running argpromotion rather than running the test through both sroa and argpromotion?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124514



More information about the llvm-commits mailing list