This is an archive of the discontinued LLVM Phabricator instance.

[ARM] Factorize the calculation of WhichResult in isV*Mask. NFC.
ClosedPublic

Authored by mstorsjo on Aug 19 2017, 12:59 PM.

Diff Detail

Event Timeline

mstorsjo created this revision.Aug 19 2017, 12:59 PM
fhahn accepted this revision.Aug 19 2017, 1:18 PM
fhahn added a subscriber: fhahn.

LGTM, I think it's a nice simplification and was already discussed in D36899. Please check the formatting before committing.

lib/Target/ARM/ARMISelLowering.cpp
5803

Line too long?

This revision is now accepted and ready to land.Aug 19 2017, 1:18 PM
mstorsjo added inline comments.Aug 19 2017, 1:23 PM
lib/Target/ARM/ARMISelLowering.cpp
5803

Indeed, fixed with clang-format-diff locally - will push with that fixed.

This revision was automatically updated to reflect the committed changes.