Page MenuHomePhabricator

Scalable type size queries (llvm)
Changes PlannedPublic

Authored by huntergr on Oct 11 2018, 6:59 AM.

Details

Summary

Implements basic size queries to support scalable vectors in LLVM IR

This isn't ready for inclusion yet, more discussion on the mailing list required.

Diff Detail

Event Timeline

huntergr created this revision.Oct 11 2018, 6:59 AM
huntergr updated this revision to Diff 172470.Nov 2 2018, 5:27 PM
  • Unified ScalableSize representation
  • Changed to uint64_t + boolean, since we no longer allow scalable vectors in aggregates
  • Removed aggregate and mixed unit tests

This looks ok to me, but I'd rather more people look at it before approving.

Thanks!

I know this isn't ready for merge, but since the mailing list discussion has died down it seems like maybe we should move the discussion here. If so, it would be helpful to have comments on all the routines explaining what they do and how they differ from the existing routines, in order to aid discussion.

simoll added a subscriber: simoll.Mar 8 2019, 9:00 AM
hsaito added a subscriber: hsaito.Mar 8 2019, 2:28 PM
huntergr planned changes to this revision.Mar 22 2019, 3:06 AM