This is an archive of the discontinued LLVM Phabricator instance.

[GlobalISel][X86] G_MUL legalizer/selector support.
ClosedPublic

Authored by igorb on May 1 2017, 2:43 AM.

Details

Summary

G_MUL legalizer/selector/regbank support.
Use only Tablegen-erated instruction selection.
This patch dealing with legal operations only.

Diff Detail

Repository
rL LLVM

Event Timeline

igorb created this revision.May 1 2017, 2:43 AM
guyblank accepted this revision.May 7 2017, 6:30 AM

LGTM

lib/Target/X86/X86LegalizerInfo.cpp
196 ↗(On Diff #97272)

can you move the vlx specific types after the hasvlx check? relevant for the function below as well.

This revision is now accepted and ready to land.May 7 2017, 6:30 AM
igorb updated this revision to Diff 98140.May 8 2017, 2:05 AM
igorb marked an inline comment as done.
  • Fixes according to the review comments.
This revision was automatically updated to reflect the committed changes.