[PATCH] D20407: [CodeGen][ObjC] zero-ext an i1 value to i8

John McCall via cfe-commits cfe-commits at lists.llvm.org
Wed May 25 16:36:01 PDT 2016


rjmccall added a comment.

In http://reviews.llvm.org/D20407#440050, @ahatanak wrote:

> In http://reviews.llvm.org/D20407#439951, @rjmccall wrote:
>
> > The C standard is poorly-written in this area, but I think it would be reasonable for CheckFunctionReturnType to just silently remove _Atomic.  (You will not be able to just re-use your new method there; removing other qualifiers is not acceptable, I'm afraid.)
>
>
> Do you mean I should fix those cases in this patch too?


No, sorry, just pointing something out for your other patch.


http://reviews.llvm.org/D20407





More information about the cfe-commits mailing list