This is an archive of the discontinued LLVM Phabricator instance.

[Polly] [PPCGCodeGeneration] Compute element size in bytes for arrays correctly.
ClosedPublic

Authored by bollu on Aug 8 2017, 4:53 AM.

Details

Diff Detail

Repository
rL LLVM

Event Timeline

bollu created this revision.Aug 8 2017, 4:53 AM
bollu updated this revision to Diff 110179.Aug 8 2017, 5:33 AM
  • [NFC] add bugpoint reduced test case
bollu updated this revision to Diff 110191.Aug 8 2017, 7:07 AM
  • [PPCGCodeGeneration] Compute element size in bytes for arrays correctly.
bollu retitled this revision from [HACK] [Polly] [PPCGCodeGeneration] Ignore Size 0 arrays during compile to [Polly] [PPCGCodeGeneration] Compute element size in bytes for arrays correctly..Aug 8 2017, 7:08 AM
bollu edited the summary of this revision. (Show Details)

Review, please.

bollu updated this revision to Diff 110192.Aug 8 2017, 7:15 AM
  • [NFC] explain why we choose to abort if size is computed to be 0
This revision is now accepted and ready to land.Aug 8 2017, 7:18 AM
grosser accepted this revision.Aug 8 2017, 7:18 AM

LGTM. Thank you!

singam-sanjay added inline comments.Aug 8 2017, 11:58 AM
test/GPGPU/array-with-elem-type-smaller-than-byte.ll
15 ↗(On Diff #110192)

const int niters

This revision was automatically updated to reflect the committed changes.