[PATCH] D140874: [clang][Interp] Support pointer types in compound assignment operations

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 18 09:46:59 PST 2023


aaron.ballman added inline comments.


================
Comment at: clang/lib/AST/Interp/ByteCodeExprGen.cpp:727-728
+    this->emitSubOffset(*RT, E);
+  else
+    return false;
+
----------------
Should this be an early return before we visit anything? Or an assert on the assumption that we should only get here for `+=` and `-=`?


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

https://reviews.llvm.org/D140874



More information about the cfe-commits mailing list