[PATCH] D96011: [NFC][Analysis] Change struct VecDesc to use ElementCount
Sander de Smalen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 11 07:27:01 PST 2021
sdesmalen accepted this revision.
sdesmalen added a comment.
This revision is now accepted and ready to land.
Thanks for the changes, LGTM.
================
Comment at: llvm/lib/Transforms/Utils/InjectTLIMappings.cpp:92-94
// All VFs in the TLI are powers of 2.
- for (unsigned VF = 2, WidestVF = TLI.getWidestVF(ScalarName); VF <= WidestVF;
- VF *= 2) {
+ ElementCount WidestFixedVF, WidestScalableVF;
+ TLI.getWidestVF(ScalarName, WidestFixedVF, WidestScalableVF);
----------------
nit: can you move this closer to the for loop?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96011/new/
https://reviews.llvm.org/D96011
More information about the llvm-commits
mailing list