This is an archive of the discontinued LLVM Phabricator instance.

[Hexagon] Return an i64 for result 0 from LowerREADCYCLECOUNTER instead of an i32.
ClosedPublic

Authored by craig.topper on Mar 19 2021, 10:00 AM.

Details

Summary

As far as I can tell, the node coming in has an i64 result so the
return should have the same type. The HexagonISD node used for
this has a type profile that says the result is i64.

Found while trying to add assserts to LegalizeDAG to catch
result type mismatches.

Diff Detail

Event Timeline

craig.topper created this revision.Mar 19 2021, 10:00 AM
craig.topper requested review of this revision.Mar 19 2021, 10:00 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 19 2021, 10:00 AM
kparzysz accepted this revision.Mar 19 2021, 10:05 AM

Thanks!

This revision is now accepted and ready to land.Mar 19 2021, 10:05 AM
This revision was landed with ongoing or failed builds.Mar 19 2021, 10:55 AM
This revision was automatically updated to reflect the committed changes.