summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJan Vorlicek <janvorli@microsoft.com>2019-03-13 01:07:00 +0100
committerJan Vorlicek <janvorli@microsoft.com>2019-03-13 01:07:00 +0100
commit78004e9a33422d1c2399339a3ac408f4c654d06b (patch)
tree929289b8ba812d044c16cde4872b9953f99e654b
parentc4ca1ddb2413a354e8f49fff4680f175a02e7d8e (diff)
downloadcoreclr-78004e9a33422d1c2399339a3ac408f4c654d06b.tar.gz
coreclr-78004e9a33422d1c2399339a3ac408f4c654d06b.tar.bz2
coreclr-78004e9a33422d1c2399339a3ac408f4c654d06b.zip
Move ANALYZER_NORETURN after function return type.
GCC doesn't like attributes before the extern keyword.
-rw-r--r--src/inc/debugmacros.h3
-rw-r--r--src/jit/error.h7
-rw-r--r--src/jit/host.h3
-rw-r--r--src/pal/src/include/pal/dbgmsg.h3
4 files changed, 6 insertions, 10 deletions
diff --git a/src/inc/debugmacros.h b/src/inc/debugmacros.h
index 655fba5e5e..540be99c74 100644
--- a/src/inc/debugmacros.h
+++ b/src/inc/debugmacros.h
@@ -30,8 +30,7 @@ bool GetStackTraceAtContext(SString & s, struct _CONTEXT * pContext);
void _cdecl DbgWriteEx(LPCTSTR szFmt, ...);
bool _DbgBreakCheck(LPCSTR szFile, int iLine, LPCSTR szExpr, BOOL fConstrained = FALSE);
-ANALYZER_NORETURN
-extern VOID DbgAssertDialog(const char *szFile, int iLine, const char *szExpr);
+extern VOID ANALYZER_NORETURN DbgAssertDialog(const char *szFile, int iLine, const char *szExpr);
#define TRACE_BUFF_SIZE (cchMaxAssertStackLevelStringLen * cfrMaxAssertStackLevels + cchMaxAssertExprLen + 1)
extern char g_szExprWithStack[TRACE_BUFF_SIZE];
diff --git a/src/jit/error.h b/src/jit/error.h
index 2a3b60fcde..18beb4b151 100644
--- a/src/jit/error.h
+++ b/src/jit/error.h
@@ -73,14 +73,13 @@ extern void DECLSPEC_NORETURN noWayAssertBody(const char* cond, const char* file
// Conditionally invoke the noway assert body. The conditional predicate is evaluated using a method on the tlsCompiler.
// If a noway_assert is hit, we ask the Compiler whether to raise an exception (i.e., conditionally raise exception.)
// To have backward compatibility between v4.5 and v4.0, in min-opts we take a shot at codegen rather than rethrow.
-ANALYZER_NORETURN
-extern void noWayAssertBodyConditional(
+extern void ANALYZER_NORETURN noWayAssertBodyConditional(
#ifdef FEATURE_TRACELOGGING
const char* file, unsigned line
#endif
);
-ANALYZER_NORETURN
-extern void noWayAssertBodyConditional(const char* cond, const char* file, unsigned line);
+
+extern void ANALYZER_NORETURN noWayAssertBodyConditional(const char* cond, const char* file, unsigned line);
// Define MEASURE_NOWAY to 1 to enable code to count and rank individual noway_assert calls by occurrence.
// These asserts would be dynamically executed, but not necessarily fail. The provides some insight into
diff --git a/src/jit/host.h b/src/jit/host.h
index 1b090a194f..436c3d1457 100644
--- a/src/jit/host.h
+++ b/src/jit/host.h
@@ -35,8 +35,7 @@ void gcDump_logf(const char* fmt, ...);
void logf(unsigned level, const char* fmt, ...);
-ANALYZER_NORETURN
-extern "C" void __cdecl assertAbort(const char* why, const char* file, unsigned line);
+extern "C" void ANALYZER_NORETURN __cdecl assertAbort(const char* why, const char* file, unsigned line);
#undef assert
#define assert(p) (void)((p) || (assertAbort(#p, __FILE__, __LINE__), 0))
diff --git a/src/pal/src/include/pal/dbgmsg.h b/src/pal/src/include/pal/dbgmsg.h
index ad3dd540cd..d7219b2bd4 100644
--- a/src/pal/src/include/pal/dbgmsg.h
+++ b/src/pal/src/include/pal/dbgmsg.h
@@ -355,8 +355,7 @@ bool DBG_ShouldCheckStackAlignment();
#else /* defined(_DEBUG) */
-ANALYZER_NORETURN
-inline void AssertBreak()
+inline void ANALYZER_NORETURN AssertBreak()
{
if(g_Dbg_asserts_enabled)
{