glandium added a comment.

In D150646#4380058 <https://reviews.llvm.org/D150646#4380058>, @mstorsjo wrote:

> Regarding "fixed" - do you see any better way of fixing it there? As it's a 
> static inline function, there's no simple way of knowing whether it was 
> already defined or not; I went with the same pattern used for the existing 
> version check for GCC.

I mean, it fixes the issue if you apply the patch or update mingw headers. 
Whether that should be required or not is a good question.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D150646/new/

https://reviews.llvm.org/D150646

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to