This is an archive of the discontinued LLVM Phabricator instance.

[Polly] Do not check JSON alignment of scalar accesses
ClosedPublic

Authored by Meinersbur on Dec 20 2015, 10:05 AM.

Details

Summary

When importing a schedule, do not verify the load/store alignment of scalar accesses. Scalar loads/store are always created newly in code generation with no alignment restrictions. Previously, scalar alignment was checked if the access instruction happened to be a LoadInst or StoreInst, but only its array (MK_Array) access is relevant.

This will be implicitly unit-tested when the access instruction of a value read can be nullptr.

Diff Detail

Event Timeline

Meinersbur retitled this revision from to [Polly] Do not check JSON alignment of scalar accesses.
Meinersbur updated this object.
Meinersbur added reviewers: grosser, jdoerfert.
Meinersbur added a project: Restricted Project.
Meinersbur added subscribers: llvm-commits, pollydev.
grosser edited edge metadata.Dec 20 2015, 10:08 AM
grosser added a subscriber: grosser.

LGTM.

jdoerfert edited edge metadata.Dec 20 2015, 12:51 PM
jdoerfert added a subscriber: jdoerfert.

Small comment

This revision was automatically updated to reflect the committed changes.