GBuella added inline comments.

================
Comment at: lib/CodeGen/CGObjCGNU.cpp:1056
+        char c = Str[i];
+        if (isalpha(c) || isnumber(c))
+          StringName += c;
----------------
theraven wrote:
> Ka-Ka wrote:
> > The isnumber() function was added to cctype.h by Apple. I don't think it 
> > can be used in llvm.
> > 
> > According to
> > https://stackoverflow.com/questions/39204080/what-is-the-difference-between-isdigit-and-isnumber
> > 
> Ah, isnumber is from 4.4BSD, I assumed it worked everywhere.  Changing it to 
> isdigit is fine.
BTW `isalnum` is in ISO since 1989.


Repository:
  rC Clang

https://reviews.llvm.org/D46052



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to