[PATCH] D145374: [FuncSpec] Consider constant struct arguments when specializing.

[Momchil Velikov via Phabricator via llvm-commits]

chill added a comment.

LGTM too, cheers!



================
Comment at: llvm/lib/Transforms/Utils/SCCPSolver.cpp:843
+
+void SCCPInstVisitor::handleSpecializationArguments(Function *F,
+                          const SmallVectorImpl<ArgInfo> &Args) {
----------------
`handleXXX` is very uninformative, it might be good in a contexts where we don't know
what kind of "handling" is done by the caller (like e.g. "visit"), but in this specific case
of setitng up/initialising lattice values for the parameters it's better called just like that, something like
`initSpecializationParemetersLaticeValues`, `setupSpecArgsValueStates`, etc.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145374