[PATCH] D64146: [ConstExprPreter][WIP] Initial patch for the constexpr interpreter
Nandor Licker via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jul 29 13:03:06 PDT 2019
nand marked 10 inline comments as done.
nand added a comment.
We can add a separate integer type which tracks all the additional information required by `__builtin_constant_p` and compile all integers to it in this context. A later patch added an APInt fallback to the interpreter if an integral cannot be mapped to a type supported by the VM - this mechanism could be used to implement the fallback for contexts which cast pointers to integers.
Comment at: clang/lib/AST/ExprVM/Compiler.h:125
+ /// Size of the local, in bytes.
+ unsigned Size;
> `ByteSize` since it's the size in bytes :)
I've removed the size field since it's not going to be used, but left the structure since it will gain other fields in future patches.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits