[llvm-dev] isspace of Visual Studio C++ assertion failed

[Michael Kruse via llvm-dev]

Hi,

I suggest to add a platform-independent version to StringExtras.h as I
did it for isprint/isPrint in r338034.

Michael


2018-07-31 0:00 GMT-05:00 Ding Fei via llvm-dev <llvm-dev at lists.llvm.org>:
> HI all,
>
> I noticed that llvm/Support/YAMLTraits.h:501
>
>     isspace(S.front() || isspace(S.back())
>
> expects non-negative chars, but our apps use non-ascii chars so it
> failed at runtime.
> On linux platform it doesn't.
>
> Would it be fixed in future release? Thanks.
>
> --
> Best Regards
>
> Ding Fei
> E-mail: danix800 at gmail.com
> _______________________________________________
> LLVM Developers mailing list
> llvm-dev at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev