This is an archive of the discontinued LLVM Phabricator instance.

[X86] Add signed aliases for popcnt intrinsics
ClosedPublic

Authored by mkuper on Dec 16 2015, 6:07 AM.

Details

Summary

The Intel manual documents both an unsigned form (_mm_popcnt_u32) and a signed form (_popcnt32) of the intrinsic.
Add the missing signed form.

Diff Detail

Repository
rL LLVM

Event Timeline

mkuper updated this revision to Diff 42996.Dec 16 2015, 6:07 AM
mkuper retitled this revision from to [X86] Add signed aliases for popcnt intrinsics.
mkuper updated this object.
mkuper added reviewers: AsafBadouh, delena.
mkuper added a subscriber: llvm-commits.
mkuper updated this revision to Diff 42999.Dec 16 2015, 6:13 AM

And now, with a test that actually checks what it ought to check.

delena accepted this revision.Dec 17 2015, 4:27 AM
delena edited edge metadata.

LGTM

This revision is now accepted and ready to land.Dec 17 2015, 4:27 AM
This revision was automatically updated to reflect the committed changes.