AuroraMiddleware/skia2
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

a little SkJumper tidy up

Browse Source 
I noticed these little bits while working on that old-Clang fix.

   - We can force-inline anytime we've got Clang,
     not just when JUMPER_IS_OFFLINE.

   - The _aarch64 and _vfp4 WRAP functions are dead code,
     as they're never compiled offline now.

Change-Id: I5850daded2ffcfe50ceeadc43f89fa8597df3387
Reviewed-on: https://skia-review.googlesource.com/84060
Commit-Queue: Mike Klein <mtklein@chromium.org>
Commit-Queue: Florin Malita <fmalita@chromium.org>
Reviewed-by: Florin Malita <fmalita@chromium.org>
This commit is contained in:
Mike Klein 2017-12-12 11:30:36 -05:00 committed by Skia Commit-Bot
parent 34adefa453
commit be22636c3d
2 changed files with 1 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/jumper/SkJumper_misc.h
View File

@ -13,7 +13,7 @@
// Miscellany used by SkJumper_stages.cpp and SkJumper_vectors.h.
// Every function in this file should be marked static and inline using SI.
#if defined(JUMPER_IS_OFFLINE)
#if defined(__clang__)
#define SI __attribute__((always_inline)) static inline
#else
#define SI static inline

4
src/jumper/SkJumper_stages.cpp
View File

@ -20,10 +20,6 @@ static const size_t N = sizeof(F) / sizeof(float);
// That lets us link together several options.
#if !defined(JUMPER_IS_OFFLINE)
#define WRAP(name) sk_##name
#elif defined(__aarch64__)
#define WRAP(name) sk_##name##_aarch64
#elif defined(__arm__)
#define WRAP(name) sk_##name##_vfp4
#elif defined(__AVX512F__)
#define WRAP(name) sk_##name##_skx
#elif defined(__AVX2__)