This is an archive of the discontinued LLVM Phabricator instance.

[PPC64][VSX] Add short, char, and bool data type for vec_vsx_ld and vec_vsx_st intrinsics
ClosedPublic

Authored by cycheng on Feb 26 2016, 12:40 AM.

Details

Summary

Issue: https://llvm.org/bugs/show_bug.cgi?id=26720

Fix compile error when building ffmpeg for PowerPC64LE because of some vec_vsx_ld/vec_vsx_st intrinsics are not supported by current clang.

New added intrinsics:

  1. (vector) {signed|unsigned} {short|char} vec_vsx_ld: (total: 8)
  2. bool vec_vsx_ld: (total: 1)
  3. (vector) {signed|unsigned} {short|char} vec_vsx_st: (total: 8)
  4. bool vec_vsx_st: (total: 1)

Total: 18 intrinsics

Diff Detail

Event Timeline

cycheng updated this revision to Diff 49153.Feb 26 2016, 12:40 AM
cycheng retitled this revision from to [PPC64][VSX] Add short, char, and bool data type for vec_vsx_ld and vec_vsx_st intrinsics.
cycheng updated this object.
cycheng added reviewers: kbarton, nemanjai, hfinkel.
cycheng added subscribers: cfe-commits, tjablin.
kbarton accepted this revision.Feb 26 2016, 2:07 PM
kbarton edited edge metadata.

LGTM.
I will commit this a bit later, when I'll be around to watch the bots.

This revision is now accepted and ready to land.Feb 26 2016, 2:07 PM

This failed testing:


Testing: 0 .. 10.. 20.. 30.. 40.. 50.. 60.. 70.. 80.. 90..

1 warning(s) in tests.
Testing Time: 225.81s


Failing Tests (1):

  Clang :: CodeGen/builtins-ppc-vsx.c

Expected Passes    : 25406
Expected Failures  : 166
Unsupported Tests  : 558
Unexpected Failures: 1

gmake[3]: * [CMakeFiles/check-all] Error 1
gmake[2]:
* [CMakeFiles/check-all.dir/all] Error 2
gmake[1]: * [CMakeFiles/check-all.dir/rule] Error 2
gmake:
* [check-all] Error 2
Test finished. Check results in /home/kbarton/llvm/build/dev.git/CHECK.log
Tests PASSED!

Reproduce with:
llvm-lit tools/clang/test/CodeGen/builtins-ppc-vsx.c

cycheng updated this revision to Diff 49335.Feb 28 2016, 6:43 PM
cycheng edited edge metadata.

Fix clang testing failure.
Thanks Kit!

I have checked the result by: make check-all
Testing result:

Testing Time: 33.36s
  Expected Passes    : 24505
  Expected Failures  : 152
  Unsupported Tests  : 365
[100%] Built target check-all
kbarton closed this revision.Mar 1 2016, 10:17 AM

Committed r262359