aaron.ballman added inline comments.
================ Comment at: clang/lib/AST/Interp/ByteCodeExprGen.cpp:213-214 + if (BO->isPtrMemOp()) + return this->visit(RHS); + ---------------- Do we need similar changes for unary operators, pointer arithmetic operators, etc? ================ Comment at: clang/lib/AST/Interp/Context.cpp:121 + if (T->isFunctionPointerType() || T->isFunctionReferenceType() || + T->isFunctionProtoType()) + return PT_FnPtr; ---------------- `isFunctionType()` in general? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144164/new/ https://reviews.llvm.org/D144164 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits