[PATCH] D18970: Add functions in ctype.h to builtin function database

Taewook Oh via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 14 08:02:31 PDT 2016


Joerg, 

Thank you for your comments. I submitted a new patch
(http://reviews.llvm.org/D19062) with a new test that checks attributes
but not types. I think this should cover both cases. What do you think?

Thanks,
Taewook

On 4/14/16, 2:46 AM, "Joerg Sonnenberger" <joerg at britannica.bec.de> wrote:

>On Wed, Apr 13, 2016 at 03:02:09PM +0000, Taewook Oh wrote:
>> Sure, I'll take a look.
>> Thanks!
>
>Please provide two versions of the test, one hard-coded for an
>architecture with signed char, one hard-coded for an architecture with
>unsigned char. That should cover the difference.
>
>Joerg



More information about the cfe-commits mailing list