Page MenuHomePhabricator

[GlobalIsel][X86] Support G_PTRTOINT instruction.
ClosedPublic

Authored by aivchenk on Feb 22 2018, 6:23 AM.

Details

Summary

Add legalization/selection for x86/x86_64 and
corresponding tests.

Diff Detail

Repository
rL LLVM

Event Timeline

aivchenk created this revision.Feb 22 2018, 6:23 AM

Hi,
Could you please add test for Legalizar pass?
I also suggest to add ll test to test all the passes together.

igorb added inline comments.Feb 27 2018, 12:42 AM
lib/Target/X86/X86InstructionSelector.cpp
691 ↗(On Diff #135409)

DEBUG(dbgs() <<TII.getName(I.getOpcode()) <<" input/output on different banks\n");

aivchenk updated this revision to Diff 136058.Feb 27 2018, 3:57 AM

addressed comments. Added legalizer and general tests

aivchenk marked an inline comment as done.Feb 27 2018, 3:57 AM
igorb accepted this revision.Feb 27 2018, 4:19 AM

LGTM,
Thanks.

This revision is now accepted and ready to land.Feb 27 2018, 4:19 AM
This revision was automatically updated to reflect the committed changes.