This is an archive of the discontinued LLVM Phabricator instance.

[X86] Make sure we still emit zext for GR32 to GR64 when the source of the zext is AssertZext
ClosedPublic

Authored by craig.topper on Sep 11 2017, 8:28 PM.

Details

Summary

The AssertZext we might see in this case is only giving information about the lower 32 bits. It isn't providing information about the upper 32 bits. So we should emit a zext.

This fixes PR28540.

Diff Detail

Repository
rL LLVM

Event Timeline

craig.topper created this revision.Sep 11 2017, 8:28 PM

Forgot to add full context to the diff the first time.

DavidKreitzer accepted this revision.Sep 18 2017, 1:29 PM

LGTM, Craig.

This revision is now accepted and ready to land.Sep 18 2017, 1:29 PM
This revision was automatically updated to reflect the committed changes.