yronglin added a comment.

In D154784#4486721 <https://reviews.llvm.org/D154784#4486721>, @rjmccall wrote:

> We had that discussion in the bug report.  Let's just increase the widths 
> unconditionally; this is not such a common expression class that we need to 
> worry about using an extra word.

Thanks for your review!

> Let's just increase the widths unconditionally

Do you mean:

  class PseudoObjectExprBitfields {
      friend class ASTStmtReader; // deserialization
      friend class PseudoObjectExpr;
  
      unsigned : NumExprBits;
      unsigned NumSubExprs : 16;
      unsigned ResultIndex : 16;
    };


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D154784/new/

https://reviews.llvm.org/D154784

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

Reply via email to