<div dir="ltr"><a href="http://llvm.org/bugs/show_bug.cgi?id=21506">http://llvm.org/bugs/show_bug.cgi?id=21506</a><br><div><br></div><div>Yep, this is what I had in mind, just kept forgetting to do it.</div><div><br></div><div>If we decide we want to move all of the code over to using this table, I think this is fine as is. If not, we should probably move the table out of the function and into src/support/classic_table.cpp.<br></div><div><br></div><div>Moving the whole of our code over to a new table is really a decision for mclow, but sgtm.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 10, 2014 at 8:58 AM, Ed Schouten <span dir="ltr"><<a href="mailto:ed@80386.nl" target="_blank">ed@80386.nl</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jonathan,<br>
<br>
Thanks for the quick response!<br>
<span class=""><br>
2014-12-10 15:46 GMT+01:00 Jonathan Roelofs <<a href="mailto:jonathan@codesourcery.com">jonathan@codesourcery.com</a>>:<br>
> Dan, didn't you file a PR saying that libc++ should just have its own table,<br>
> and use that for everything?<br>
<br>
</span>That would be nice! One of the things we could then consider doing, is<br>
declaring the table as having 256 elements. Right now we have quite a<br>
lot of functions in libc++ that do something like:<br>
<br>
bool isalpha(char c) {<br>
  return isascii(c) ? (classic_table[c] & alpha) != 0 : false;<br>
}<br>
<br>
The problem with this approach is that isascii() is an obsolete X/Open<br>
system interfaces function.<br>
<br>
The code could be changed to manually test whether the value is <=<br>
127, but if the classic_table would grow to 256, there shouldn't even<br>
be a reason to do the bounds checking.<br>
<br>
Thoughts?<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Ed Schouten <<a href="mailto:ed@80386.nl">ed@80386.nl</a>><br>
</font></span></blockquote></div><br></div>