[PATCH] D152281: [Transforms][LICM] Add the ability to undo unprofitable reassociation

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm accepted this revision.
paulwalker-arm added a comment.

Thanks for adding the command line option @pawosm01.



================
Comment at: llvm/lib/Transforms/Scalar/LICM.cpp:133
+cl::opt<unsigned> FPAssociationUpperLimit(
+    "licm-fpassociation-upper-limit", cl::init(5U), cl::Hidden,
+    cl::desc(
----------------
Up to you but I think "licm-max-num-fp-reassociations" or "licm-fp-reassociation-cap" would be more in keeping with the existing naming of licm options.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152281