jlebar added a subscriber: jlebar.
jlebar added a comment.

There seem to be many nontrivial whitespace errors introduced by this patch.  
For example,

  -        Attrs.push_back(HTMLStartTagComment::Attribute(Ident.getLocation(),
  -                                                       
Ident.getHTMLIdent()));
  +        Attrs.emplace_back(Ident.getLocation(),
  +                                                       Ident.getHTMLIdent());

and

  -        BitGroups.push_back(BitGroup(LastValue, LastRLAmt, LastGroupStartIdx,
  -                                     i-1));
  +        BitGroups.emplace_back(LastValue, LastRLAmt, LastGroupStartIdx,
  +                                     i-1);


Repository:
  rL LLVM

http://reviews.llvm.org/D21507



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to