[PATCH] D150364: [clang][Interp] Add 'Invalid' opcode and use it for throw/asm stmts

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

tbaeder added inline comments.


================
Comment at: clang/test/AST/Interp/records.cpp:341
+
+    /// FIXME: Wrong source location below.
+    return 12; // expected-note {{in call to '&S{}->~S()'}}
----------------
aaron.ballman wrote:
> Oh interesting -- does the old constexpr interpreter think the destructor is called at the end of the block as opposed to at the end of the full expression with the temporary?
I think it's the other way around and the new one does that (`expected` is the new one). Need to investigate that.


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

https://reviews.llvm.org/D150364