LGTM, you could probably just do metadata {{.*}}, metadata !"bad_carrier", .... to get rid of some of your check lines.<div><br></div><div>Couple of stray whitespace changes that make dgregor grumpy though :)<div>
<br></div><div>-eric<br><br><div class="gmail_quote">On Wed, Feb 20, 2013 at 3:20 PM, Adrian Prantl <span dir="ltr"><<a href="mailto:aprantl@apple.com" target="_blank">aprantl@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi CFEes,<br>
<br>
please review the following patch:<br>
<br>
Emit the DIType for ObjC 'id<protocol>' as 'id' rather than objc_object*<br>
Fix for rdar://problem/12601916.<br>
<br>
The frontend treats 'id' as a typedef to an ObjCObjectType,<br>
whereas 'id<protocol>' is treated as an ObjCPointerType. For the<br>
debug info, we want to emit 'id' in both cases.<br>
<br>
cheers,<br>
Adrian<br>
<br>
</blockquote></div><br></div></div>