[PATCH] D149768: [ArgumentPromotion] Bail if any callers are minsize and more instructions are added than removed

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 9 14:56:39 PDT 2023


aeubanks added a comment.

In D149768#4330784 <https://reviews.llvm.org/D149768#4330784>, @smeenai wrote:

> Thanks for the update! I'm working on testing the size change with the new revision, but our internal infra for that is a little broken right now, so it might take a bit. I'm also still struggling with llvm-reduce's tendency to introduce `unreachable` in reachable places and use uninitialized memory, unfortunately.

No rush :)

I'm not sure how you're trying using llvm-reduce, it's not meant for any meaningful semantics-preserving transformations.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149768



More information about the llvm-commits mailing list