<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Oct 2, 2017 at 4:25 PM, Rafael Avila de Espindola via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Reid Kleckner via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> writes:<br>
<br>
> rnk added a comment.<br>
><br>
> cppreference suggests writing our own wrapper to avoid this issue: <a href="http://en.cppreference.com/w/cpp/string/byte/isdigit" rel="noreferrer" target="_blank">http://en.cppreference.com/w/<wbr>cpp/string/byte/isdigit</a><br>
><br>
> I'd suggest adding `llvm::isDigit` and `llvm::isHexDigit` in StringExtras.h next to our other hex digit conversion routines, so we don't have to repeat this surprising cast everywhere.<br>
<br>
</span>I agree.<br></blockquote><div><br></div><div>Please add llvm::isAlpha and llvm::isAlnum too. We use these functions in lld.</div></div></div></div>