Page MenuHomePhabricator

[mlir] Update several usages of IntegerType to properly handled unsignedness.
ClosedPublic

Authored by rriddle on Feb 28 2020, 11:09 AM.

Details

Summary

For example, DenseElementsAttr currently does not properly round-trip unsigned integer values.

Diff Detail

Event Timeline

rriddle created this revision.Feb 28 2020, 11:09 AM
antiagainst accepted this revision.Mar 2 2020, 8:30 AM

Nice. Thanks River! I've a CL to add more ODS definitions for integer signedness but haven't pushed out. Will do that soon.

This revision is now accepted and ready to land.Mar 2 2020, 8:30 AM
This revision was automatically updated to reflect the committed changes.