erichkeane added inline comments.
================ Comment at: clang/lib/AST/Interp/ByteCodeExprGen.cpp:289 + QualType ArgType = E->getTypeOfArgument(); + return this->emitConst(E, Ctx.getASTContext().getTypeSize(ArgType)); + } ---------------- You probably want `getTypeSizeInChars`. `getTypeSize` returns it in bits, so this would give the wrong answer. You should probably add some tests to make sure the values you are getting back are sane (like `char` is 1, etc). CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133934/new/ https://reviews.llvm.org/D133934 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
