[PATCH] D33695: [Solaris] Fix PR33228 - llvm::sys::fs::is_local_impl done right

[Fedor Sergeev via Phabricator via llvm-commits]

fedor.sergeev added a comment.

In https://reviews.llvm.org/D33695#768261, @krytarowski wrote:

> Illumos uses `__sun`, not `__sun__`. I know that `__something__` is more POSIX friendly, but does it handle only commercial Solaris? What version?


gcc on Solaris 10/11 sparc/intel specifies both __sun and __sun__.
I can see __sun being used in llvm sources once, but none of __sun__.
I will switch to __sun.


https://reviews.llvm.org/D33695