================
@@ -28,11 +28,26 @@ class Block;
class DeadBlock;
class Pointer;
class Context;
+template <unsigned A, bool B> class Integral;
enum PrimType : unsigned;
class Pointer;
inline llvm::raw_ostream &operator<<(llvm::raw_ostream &OS, const Pointer &P);
+struct BlockPointer {
----------------
AaronBallman wrote:
This is not the first time I've hit this cognitive issue, but `Block` is an
existing term of art in the compiler and means something totally different from
how we use it in the interpreter. My very first thought was that this was a
significant amount of work just to support Objective-C block pointers, why
aren't we doing the same thing for member pointers, etc. It took me a while to
realize you mean a descriptor for a chunk of memory.
I don't know if others reviewing the interpreter have hit similar issues with
"block" or not, so I'm not recommending any changes. But it is something to
keep in mind when naming stuff.
https://github.com/llvm/llvm-project/pull/84159
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits