[PATCH] D135989: [clang][Sema] Use size of char for void types

Bill Wendling via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 14 14:12:53 PDT 2022


void added inline comments.


================
Comment at: clang/test/Sema/array-bounds-ptr-arith.c:14
+void* broken (struct ext2_super_block *es,int a) {
+  return (void *)es->s_uuid + 9; // expected-warning {{the pointer incremented by 9 refers past the end of the array (that contains 8 elements)}}
 }
----------------
serge-sans-paille wrote:
> I'm fine with that change, but I don't understand how it relates to that commit ;-)
The original size 80 was large enough to emit a warning. However, 9 *didn't* emit a warning, but should. (Note that 8 won't emit a warning because of pointer arithmetic voodoo.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135989



More information about the cfe-commits mailing list