Ok, I now reuse Sema::RecordDeclSetTy.<div><br></div><div>As for putting a bit into FieldDecl as opposed to the UnusedPrivateField SetVector: It seems like FieldDecl is carefully crafted for size. Just adding another bit for this use case doesn't seem right.</div>
<div><br></div><div>Please find the new version attached.<br><br><div class="gmail_quote">On Wed, May 9, 2012 at 5:41 PM, Jordan Rose <span dir="ltr"><<a href="mailto:jediknil@belkadan.com" target="_blank">jediknil@belkadan.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><br>
On May 9, 2012, at 19:24, Daniel Jasper wrote:<br>
<br>
>> - IsRecordFullyDefined should probably use SmallPtrSetImpl, to avoid specifying a size in the function prototype. (Not that it /really/ matters, but...)<br>
>><br>
> But all the methods in SmallPtrSetImpl are protected!?<br>
<br>
</div>Oops, sorry. I didn't actually check the docs; I just assumed it was like SmallVector and SmallVectorImpl. I suppose a typedef, then, is the next best thing, to avoid having to specify the count in two different places.</blockquote>
</div><br></div>