[PATCH] D56957: GlobalISel: Handle some odd splits in fewerElementsVector
Daniel Sanders via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 29 14:46:56 PDT 2019
dsanders added inline comments.
================
Comment at: lib/CodeGen/GlobalISel/LegalizerHelper.cpp:1398
+ for (unsigned Offset = 0; Offset < BitsForNumParts; Offset += NarrowSize) {
+ SmallVector<SrcOp, 4> SrcOps;
+ for (unsigned I = 1, E = MI.getNumOperands(); I != E; ++I) {
----------------
Hi Matt,
I just came across the various uses of SrcOp in LegalizerHelper and thought I should mention that this type wasn't really meant to be used outside of MachineIRBuilder and its specializations. It's a bit of machinery to be able to factor out the common overloads.
What was the reason for introducing the SrcOp variables? Does MachineOperand or registers achieve the same end result?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56957/new/
https://reviews.llvm.org/D56957
More information about the llvm-commits
mailing list