Skip to content

S
swiftshader
Commit 0571621b by Tom Tan
Options

Port LLVM change rL350655

Chromium enabled warning -Wmicrosoft-include which complains the line "#include "Windows/WindowsSupport.h"" in lib/Support/Windows/Threading.inc because both files are under the same folder (see below error message). The fix was merged to Clang master at https://reviews.llvm.org/rL350655. Port the change to unblock Windows ARM64 build before a new rolling of LLVM to swiftshader. error: #include resolved using non-portable Microsoft search rules as: ../../third_party/swiftshader/third_party/llvm-7.0/llvm/lib/Support/Windows/WindowsSupport.h [-Werror,-Wmicrosoft-include] Bug: chromium:893460 Change-Id: I9eb32fd92e4d96f7d22a07a7c2da70b98eba19d3 Reviewed-on: https://swiftshader-review.googlesource.com/c/23669Reviewed-by: 's avatarAlexis Hétu <sugoi@google.com> Tested-by: 's avatarTom Tan <Tom.Tan@microsoft.com>
parent c25b8077
Showing with 1 addition and 1 deletion
third_party/llvm-7.0/llvm/lib/Support/Windows/Threading.inc
...@@ -14,7 +14,7 @@ ...@@ -14,7 +14,7 @@
#include "llvm/ADT/SmallString.h" #include "llvm/ADT/SmallString.h"
#include "llvm/ADT/Twine.h" #include "llvm/ADT/Twine.h"
#include "Windows/WindowsSupport.h" #include "WindowsSupport.h"
#include <process.h> #include <process.h>
// Windows will at times define MemoryFence. // Windows will at times define MemoryFence.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment