[PATCH] D32681: [builtins] Enabled emulated TLS on WOA.

NAKAMURA Takumi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 1 17:55:35 PDT 2017


chapuni added a comment.

In https://reviews.llvm.org/D32681#742912, @compnerd wrote:

> I have a local patch (well, it's either on reviews.llvm.org or on the mailing list as well) that emulates the case insensitivity, but that is expensive, and I'd rather keep the hit to the actual file when possible.  How about adding a guard around the incorrect case (yes, Microsoft doesn't get the case right, but the SDK provides the file with the capital name, so that should be the official version IMO).


I really wish you to stop following its capital file names. <windows.h> should be fine.


Repository:
  rL LLVM

https://reviews.llvm.org/D32681





More information about the llvm-commits mailing list