This is an archive of the discontinued LLVM Phabricator instance.

Add support for scalarizing cttz_zero_undef
ClosedPublic

Authored by petarj on Aug 6 2014, 11:45 AM.

Details

Summary

Follow up to r214266. Add missing case in ScalarizeVectorResult() for
cttz_zero_undef.

Diff Detail

Repository
rL LLVM

Event Timeline

petarj updated this revision to Diff 12244.Aug 6 2014, 11:45 AM
petarj retitled this revision from to Add support for scalarizing cttz_zero_undef.
petarj updated this object.
petarj edited the test plan for this revision. (Show Details)
petarj added reviewers: arsenm, chandlerc.
petarj set the repository for this revision to rL LLVM.
petarj added a project: deleted.
petarj added a subscriber: Unknown Object (MLST).
arsenm edited edge metadata.Aug 7 2014, 10:56 AM

The actual patch part is obvious. The test looks overly strict, with the specific registers used for each instruction

petarj updated this revision to Diff 12293.Aug 7 2014, 3:01 PM
petarj edited edge metadata.

I have loosened up the restrictions in the test case.
PTAL.

arsenm accepted this revision.Aug 8 2014, 12:14 PM
arsenm edited edge metadata.

LGTM

This revision is now accepted and ready to land.Aug 8 2014, 12:14 PM
petarj closed this revision.Aug 10 2014, 3:59 PM
petarj updated this revision to Diff 12338.

Closed by commit rL215330 (authored by @petarj).