shafik added inline comments.
================ Comment at: clang/lib/AST/Interp/Interp.h:970 + if (!Pointer::hasSameArray(LHS, RHS)) { + // TODO: Diagnose. + return false; ---------------- tbaeder wrote: > This is also not being diagnosed (only rejected) by the current interpreter. > But would be nice to have an error message. The TODO is still present, will you tackle this as part of this PR or a follow-up? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D135858/new/ https://reviews.llvm.org/D135858 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits