This is an archive of the discontinued LLVM Phabricator instance.

GlobalISel: Add LLT helper to multiply vector sizes
ClosedPublic

Authored by arsenm on Apr 11 2022, 4:23 PM.

Diff Detail

Event Timeline

arsenm created this revision.Apr 11 2022, 4:23 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 11 2022, 4:23 PM
arsenm requested review of this revision.Apr 11 2022, 4:23 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 11 2022, 4:23 PM
Herald added a subscriber: wdng. · View Herald Transcript
paquette added inline comments.Apr 15 2022, 1:30 PM
llvm/include/llvm/Support/LowLevelTypeImpl.h
213

why int?

arsenm added inline comments.Apr 15 2022, 1:38 PM
llvm/include/llvm/Support/LowLevelTypeImpl.h
213

Copy paste from the above divide. Plus I'm generally of the belief that using unsigned is bad, but basically everywhere else uses unsigned

paquette accepted this revision.Apr 15 2022, 1:40 PM

LGTM

llvm/include/llvm/Support/LowLevelTypeImpl.h
213

that's like the opposite of my personal belief

This revision is now accepted and ready to land.Apr 15 2022, 1:40 PM