This is an archive of the discontinued LLVM Phabricator instance.

[GlobalISel]: Fix a bug in KnownBits where we should have been using SizeInBits
ClosedPublic

Authored by aditya_nandakumar on Aug 9 2019, 4:25 PM.

Details

Summary

We were using getIndexSize instead of getIndexSizeInBits().
Added test case for G_PTRTOINT and G_INTTOPTR.

Diff Detail

Repository
rL LLVM

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptAug 9 2019, 4:25 PM
bogner accepted this revision.Aug 12 2019, 1:53 PM
bogner added a subscriber: bogner.

Obvious bugfixes like this can generally be post-commit reviewed.

This revision is now accepted and ready to land.Aug 12 2019, 1:53 PM
volkan accepted this revision.Aug 12 2019, 1:53 PM

LGTM.