patch to fix Expr::isIntegerConstantExpr() for sizeof

Hi,

Attached is a patch to fix Expr::isIntegerConstantExpr() for a simple "sizeof(int)". This problem is the same as the one I sent before, but it seems clang has 2 ways to represent the sizeof operator (as a unary operator and as a sizeof operator) and thus the code/bug is duplicated.

Regards,
Nuno

clang_sizeof_patch.txt (748 Bytes)

Thanks. Applied! (the only change I made was that I wrapped a couple lines at 80 columns).

Hi,

Attached is a patch to fix Expr::isIntegerConstantExpr() for a simple "sizeof(int)". This problem is the same as the one I sent before, but it seems clang has 2 ways to represent the sizeof operator (as a unary operator and as a sizeof operator) and thus the code/bug is duplicated.

Regards,
Nuno

Thanks. Applied! (the only change I made was that I wrapped a couple lines at 80 columns).

Thanks! (I need to get an editor that shows the 80-column mark..)
Nuno