Page MenuHomePhabricator

Added tests for lowerInterleavedStore.
ClosedPublic

Authored by Farhana on May 30 2017, 9:43 AM.

Details

Summary

This change-set added few tests for testing a set of interleaved stores. It tests the currently generated by LLVM for a set of interleaved stores.

Diff Detail

Repository
rL LLVM

Event Timeline

Farhana created this revision.May 30 2017, 9:43 AM
Farhana added reviewers: RKSimon, DavidKreitzer.

Hi Simon,

I've created a separate change-set only with the tests following your request (below) in https://reviews.llvm.org/D32658.

RKSimon added a comment.
Fri, May 5, 5:18 AM
Comment Actions
Please can you add the new tests to trunk now with the current codegen so your patch shows the diff.

Farhana

RKSimon edited edge metadata.May 30 2017, 10:08 AM

What happened to test/CodeGen/X86/x86-interleaved-access.ll ?

Farhana updated this revision to Diff 100728.May 30 2017, 10:20 AM

updated x86-interleaved-access.ll

What happened to test/CodeGen/X86/x86-interleaved-access.ll ?

What happened to test/CodeGen/X86/x86-interleaved-access.ll ?

Sorry, something went wrong with my uploading earlier. I uploaded the tests in x86-interleaved-access.ll again.

Farhana updated this revision to Diff 100747.May 30 2017, 12:24 PM

Thanks, Farhana. Please remember to add llvm-commits as a subscriber for LLVM patches. This LGTM unless Simon has additional suggestions.

RKSimon accepted this revision.May 30 2017, 2:19 PM

LGTM

This revision is now accepted and ready to land.May 30 2017, 2:19 PM
This revision was automatically updated to reflect the committed changes.