tbaeder added inline comments.
================ Comment at: clang/lib/AST/Interp/Interp.h:970 + if (!Pointer::hasSameArray(LHS, RHS)) { + // TODO: Diagnose. + return false; ---------------- shafik wrote: > 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? A follow-up would be better I think. 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