Re: r281351 - Add a class ObjCProtocolQualifiers to wrap APIs for ObjC protocol list.

2016-09-13 Thread Artem Belevich via cfe-commits
Thanks for the quick fix. ASAN is happy now. --Artem On Tue, Sep 13, 2016 at 3:09 PM, Manman wrote: > I checked in r281404. Hopefully it will fix the issue. > > Let me know if it does not. > > Thanks, > Manman > > On Sep 13, 2016, at 3:03 PM, Artem Belevich via cfe-commits < >

Re: r281351 - Add a class ObjCProtocolQualifiers to wrap APIs for ObjC protocol list.

2016-09-13 Thread Manman via cfe-commits
I checked in r281404. Hopefully it will fix the issue. Let me know if it does not. Thanks, Manman > On Sep 13, 2016, at 3:03 PM, Artem Belevich via cfe-commits > wrote: > > Manman, > > FYI, It appears that some of your ObjC commits today trigger asan error. >

Re: r281351 - Add a class ObjCProtocolQualifiers to wrap APIs for ObjC protocol list.

2016-09-13 Thread Artem Belevich via cfe-commits
Manman, FYI, It appears that some of your ObjC commits today trigger asan error. Sanitizer bots are broken by PR30341, so they don't report the issue yet. --Artem $ llvm/tools/clang/clang -cc1 -internal-isystem llvm/tools/clang/staging/include -nostdsysteminc -fblocks -fsyntax-only

r281351 - Add a class ObjCProtocolQualifiers to wrap APIs for ObjC protocol list.

2016-09-13 Thread Manman Ren via cfe-commits
Author: mren Date: Tue Sep 13 11:45:29 2016 New Revision: 281351 URL: http://llvm.org/viewvc/llvm-project?rev=281351=rev Log: Add a class ObjCProtocolQualifiers to wrap APIs for ObjC protocol list. Now ObjCObjectType extends from ObjCProtocolQualifiers. We save number of protocols in