[PATCH] D144164: [clang][Interp] Handle PtrMemOps

[Timm Bäder via Phabricator via cfe-commits]

tbaeder added inline comments.


================
Comment at: clang/lib/AST/Interp/ByteCodeExprGen.cpp:234
 
+  if (!LT || !RT || !T)
+    return this->bail(BO);
----------------
aaron.ballman wrote:
> Should we be checking `!T` earlier (it's used within the `Discard` lambda with an unprotected dereference)?
Probably, but I changed this again a bit in https://reviews.llvm.org/D148925, so I'd like to do that change in a later commit.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144164/new/

https://reviews.llvm.org/D144164