maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Define Neon intrinsics as "static inline" to avoid warning. rdar://13108414 

We had been defining Neon intrinsics as "static" with always_inline attributes.
If you use them from an extern inline function, you get a warning, e.g.:

static function 'vadd_u8' is used in an inline function with external linkage

This change simply adds the inline keyword to avoid that warning.

llvm-svn: 179406
This commit is contained in:
Bob Wilson 2013-04-12 20:17:20 +00:00
parent b3ae876174
commit 2b59395d0e
2 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
clang/test/Sema/arm-neon-types.c
View File

@ -1,4 +1,5 @@
// RUN: %clang_cc1 -triple thumbv7-apple-darwin10 -target-cpu cortex-a8 -fsyntax-only -Wvector-conversion -ffreestanding -verify %s
#ifndef INCLUDE
#include <arm_neon.h>
@ -33,3 +34,14 @@ int16x8_t test5(int *p) {
void test6(float *p, int32x2_t v) {
return vst1_s32(p, v); // expected-warning {{incompatible pointer types}}
}
#define INCLUDE
#include "arm-neon-types.c"
#else
// Make sure we don't get a warning about using a static function in an
// extern inline function from a header.
extern inline uint8x8_t test7(uint8x8_t a, uint8x8_t b) {
return vadd_u8(a, b);
}
#endif

2
clang/utils/TableGen/NeonEmitter.cpp
View File

@ -1342,7 +1342,7 @@ void NeonEmitter::run(raw_ostream &OS) {
}
}
OS<<"#define __ai static __attribute__((__always_inline__, __nodebug__))\n\n";
OS<<"#define __ai static inline __attribute__((__always_inline__, __nodebug__))\n\n";
std::vector<Record*> RV = Records.getAllDerivedDefinitions("Inst");