Page MenuHomePhabricator

statepoint-call-lowering: relocate array and vector
AbandonedPublic

Authored by artagnon on Jan 12 2015, 2:57 PM.

Details

Reviewers
reames
sanjoy
Summary

This is mainly to present the direction in which future work
is going, and gather early feedback.

Diff Detail

Event Timeline

artagnon updated this revision to Diff 18051.Jan 12 2015, 2:57 PM
artagnon retitled this revision from to statepoint-call-lowering: relocate array and vector.
artagnon updated this object.
artagnon edited the test plan for this revision. (Show Details)
artagnon added reviewers: reames, sanjoy.
artagnon added a subscriber: Unknown Object (MLST).
reames edited edge metadata.Jan 12 2015, 4:24 PM

This looks like it overlaps with http://reviews.llvm.org/D6915?

This looks like it overlaps with http://reviews.llvm.org/D6915?

Yes, it incidentally does: D6915 is meant to test that the name mangling happens correctly, and incidentally uses gc.statepoint, gc.relocate to achieve its ends. This is specifically about testing the assembly of lowering.

Can you give me a hint about how vector relocations are supposed to look in assembly? I aim to include changes to SelectionDAG/StatepointLowering with this patch: the first step is to figure out what is wrong with the lowering.

artagnon abandoned this revision.Jan 14 2015, 4:44 PM

This conflates two different things:

  1. Adding vector support to gc.relocate.
  2. Adding iAny array support for intrinsics.

Expect two different review requests.