efriedma added a comment.
As a practical matter, I'm not sure this helps much; ExprConstant should be
reasonably fast for simple integers. The performance issues only really show
for structs/arrays. But maybe it helps a little to handle a few of the most
common integer expressions.
================
Comment at: clang/lib/CodeGen/CGExprConstant.cpp:1128
case CK_ConstructorConversion:
+ case CK_IntegralCast:
return Visit(subExpr, destType);
----------------
This doesn't look right; I suspect this will return an integer with the wrong
width.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156154/new/
https://reviews.llvm.org/D156154
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
