[PATCH] D156400: [clang][Interp] Implement __builtin_offsetof

Corentin Jabot via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 3 05:05:22 PDT 2023


cor3ntin added inline comments.


================
Comment at: clang/lib/AST/Interp/ByteCodeExprGen.cpp:1536-1537
+      // Cast to Sint64.
+      if (IndexT != PT_Sint64) {
+        if (!this->emitCast(IndexT, PT_Sint64, E))
+          return false;
----------------
Shouldn't that be size_t?


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

https://reviews.llvm.org/D156400



More information about the cfe-commits mailing list