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

Saleem Abdulrasool via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 1 17:41:48 PDT 2017


compnerd added a comment.

@mstorsjo Yeah, clang supports proper TLS on Windows on ARM.  This is only needed if you are explicitly going out of your way to use the emulated TLS (via `-femulated-tls`).  Otherwise, this code path shouldn't be hit.

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).


Repository:
  rL LLVM

https://reviews.llvm.org/D32681





More information about the llvm-commits mailing list