[PATCH] D79810: [FileCheck] Fix isalpha/isalnum calls

Joel E. Denny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 14 11:23:53 PDT 2020


jdenny marked an inline comment as done.
jdenny added a comment.

In D79810#2034482 <https://reviews.llvm.org/D79810#2034482>, @efriedma wrote:

> Please use llvm::isAlpha/llvm::isAlnum instead of the functions from ctype.h; ctype.h is locale-dependent.


Thanks!  I wasn't aware of those.


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

https://reviews.llvm.org/D79810





More information about the llvm-commits mailing list