Attachment 350007 Details for Bug 171618 – WIP Patch to define WTF_COMPILER_MSVC

[patch] WIP Patch to define WTF_COMPILER_MSVC

wip.patch (text/plain), 2.81 KB, created by Fujii Hironori on 2018-09-17 23:42:43 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Fujii Hironori

Created: 2018-09-17 23:42:43 PDT

Size: 2.81 KB

patch

obsolete

>diff --git a/Source/JavaScriptCore/jit/JITMathIC.h b/Source/JavaScriptCore/jit/JITMathIC.h
>index 4e00958d19f..68167926fba 100644
>--- a/Source/JavaScriptCore/jit/JITMathIC.h
>+++ b/Source/JavaScriptCore/jit/JITMathIC.h
>@@ -144,7 +144,7 @@ public:
>         };
> 
>         auto replaceCall = [&] () {
>-#if COMPILER(MSVC)
>+#if COMPILER(MSVC) && !COMPILER(CLANG)
>             ftlThunkAwareRepatchCall(codeBlock, slowPathCallLocation().retagged<JSInternalPtrTag>(), callReplacement);
> #else
>             ftlThunkAwareRepatchCall(codeBlock, slowPathCallLocation().template retagged<JSInternalPtrTag>(), callReplacement);
>diff --git a/Source/WTF/wtf/Compiler.h b/Source/WTF/wtf/Compiler.h
>index 71e6b8e6a08..5a921d32120 100644
>--- a/Source/WTF/wtf/Compiler.h
>+++ b/Source/WTF/wtf/Compiler.h
>@@ -363,7 +363,7 @@
> 
> /* UNUSED_PARAM */
> 
>-#if !defined(UNUSED_PARAM) && COMPILER(MSVC)
>+#if !defined(UNUSED_PARAM) && COMPILER(MSVC) && !COMPILER(CLANG)
> #define UNUSED_PARAM(variable) (void)&variable
> #endif
> 
>@@ -430,7 +430,7 @@
> #define IGNORE_GCC_WARNINGS_END
> #endif
> 
>-#if COMPILER(CLANG)
>+#if COMPILER(CLANG) && !COMPILER(MSVC)
> #define IGNORE_CLANG_WARNINGS_BEGIN(warning) IGNORE_WARNINGS_BEGIN_IMPL(clang, warning)
> #define IGNORE_CLANG_WARNINGS_END IGNORE_WARNINGS_END_IMPL(clang)
> #else
>diff --git a/Source/cmake/WebKitCompilerFlags.cmake b/Source/cmake/WebKitCompilerFlags.cmake
>index 72dc5c96779..6bd1ff7fa69 100644
>--- a/Source/cmake/WebKitCompilerFlags.cmake
>+++ b/Source/cmake/WebKitCompilerFlags.cmake
>@@ -116,18 +116,17 @@ if (COMPILER_IS_GCC_OR_CLANG)
>             WEBKIT_PREPEND_GLOBAL_COMPILER_FLAGS(-Wno-unknown-pragmas)
>             add_definitions(-D__USE_MINGW_ANSI_STDIO=1)
>         endif ()
>+        # Warnings to be enabled
>+        WEBKIT_PREPEND_GLOBAL_COMPILER_FLAGS(-Wall
>+                                             -Wextra
>+                                             -Wcast-align
>+                                             -Wformat-security
>+                                             -Wmissing-format-attribute
>+                                             -Wpointer-arith
>+                                             -Wundef
>+                                             -Wwrite-strings)
>     endif ()
> 
>-    # Warnings to be enabled
>-    WEBKIT_PREPEND_GLOBAL_COMPILER_FLAGS(-Wall
>-                                         -Wextra
>-                                         -Wcast-align
>-                                         -Wformat-security
>-                                         -Wmissing-format-attribute
>-                                         -Wpointer-arith
>-                                         -Wundef
>-                                         -Wwrite-strings)
>-
>     # Warnings to be disabled
>     # FIXME: We should probably not be disabling -Wno-maybe-uninitialized?
>     WEBKIT_PREPEND_GLOBAL_COMPILER_FLAGS(-Qunused-arguments

Actions: View | Formatted Diff | Diff

Attachments on bug 171618: 349006 | 349010 | 349627 | 350006 | 350007 | 467870