[clang] Patch for 6037
Richard Smith
richard at metafoo.co.uk
Fri Jan 16 17:24:32 PST 2015
On Fri, Jan 16, 2015 at 11:49 AM, Nathan Sidwell <nathan at acm.org> wrote:
> On 01/14/15 23:11, Richard Smith wrote:
>>
>> +typedef std::set<QualType, QualTypeOrdering> IndirectBaseSet;
>>
>> Use llvm::SmallPtrSet rather than std::set here. Other than that, just
>> some typographical comments:
>
>
> Thanks for review. This version moves to a SmallPtrSet and addresses your
> other comments. I hope.
>
> ok?
LGTM. Please convert tabs to spaces prior to commit (there are about
half a dozen lines with hard tabs in this patch), and put this { on
the previous line:
+NoteIndirectBases(ASTContext &Context, IndirectBaseSet &Set,
+ const QualType &Type)
+{
More information about the cfe-commits
mailing list