Page MenuHomePhabricator

[SVE] Remove usages of VectorType::getNumElements() from ARM
ClosedPublic

Authored by ctetreau on Tue, May 12, 3:26 PM.

Diff Detail

Event Timeline

ctetreau created this revision.Tue, May 12, 3:26 PM
fpetrogalli added inline comments.Wed, May 13, 6:38 AM
llvm/lib/Target/ARM/MVETailPredication.cpp
348

The RHS is not something along the lines of VectorType::get(), or an explicit cast. Please consider reverting it.

dmgreen accepted this revision.Wed, May 13, 6:44 AM

Sounds good to me. +1 to @fpetrogalli's comment.

This revision is now accepted and ready to land.Wed, May 13, 6:44 AM
ctetreau updated this revision to Diff 264066.Thu, May 14, 12:51 PM

address code review issues

ctetreau updated this revision to Diff 264310.Fri, May 15, 12:27 PM

Fix whatever happened at the top of MVETailPredication, use the correct vector type at the bottom

ctetreau updated this revision to Diff 264312.Fri, May 15, 12:30 PM

fix file header

This revision was automatically updated to reflect the committed changes.