[PATCH] D124178: [ArgPromotion] Change the condition to check the promotion limit

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 21 11:57:31 PDT 2022


aeubanks accepted this revision.
aeubanks added a comment.
This revision is now accepted and ready to land.

looks good



================
Comment at: llvm/test/Transforms/ArgumentPromotion/max-elements-limit.ll:1
+; RUN: opt -passes=argpromotion -S -o - %s | FileCheck %s
+
----------------
`-o -` should be unnecessary


================
Comment at: llvm/test/Transforms/ArgumentPromotion/max-elements-limit.ll:52
+
+!llvm.module.flags = !{!0, !1, !2}
+
----------------
remove unrelated IR (and things like `dso_local` above)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124178



More information about the llvm-commits mailing list