joechrisellis created this revision. joechrisellis added reviewers: DavidTruby, peterwaller-arm, paulwalker-arm, bsmith. Herald added subscribers: psnobl, kristof.beyls, tschuett. Herald added a reviewer: efriedma. joechrisellis requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits.
The inline keyword is not defined in the C89 standard, so source files that include arm_sve.h will fail compilation if -std=c89 is specified. We already use the __always_inline__ function attribute so the inline keyword here is redundant anyway. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D96852 Files: clang/utils/TableGen/SveEmitter.cpp Index: clang/utils/TableGen/SveEmitter.cpp =================================================================== --- clang/utils/TableGen/SveEmitter.cpp +++ clang/utils/TableGen/SveEmitter.cpp @@ -1201,7 +1201,7 @@ OS << "};\n\n"; OS << "/* Function attributes */\n"; - OS << "#define __aio static inline __attribute__((__always_inline__, " + OS << "#define __aio static __attribute__((__always_inline__, " "__nodebug__, __overloadable__))\n\n"; // Add reinterpret functions.
Index: clang/utils/TableGen/SveEmitter.cpp =================================================================== --- clang/utils/TableGen/SveEmitter.cpp +++ clang/utils/TableGen/SveEmitter.cpp @@ -1201,7 +1201,7 @@ OS << "};\n\n"; OS << "/* Function attributes */\n"; - OS << "#define __aio static inline __attribute__((__always_inline__, " + OS << "#define __aio static __attribute__((__always_inline__, " "__nodebug__, __overloadable__))\n\n"; // Add reinterpret functions.
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits