[PATCH] D157331: [clang] Implement C23 <stdckdint.h>

Zijun Zhao via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 30 11:03:07 PDT 2023


ZijunZhao updated this revision to Diff 554784.
ZijunZhao marked 3 inline comments as done.
ZijunZhao added a comment.

Remove the short type check part.  
>From Aaron Ballman:

> Instead, we'd want to do some data flow analysis (in the clang static analyzer) so we know the potential range of values of each of the operands and can diagnose the result type from there if we can prove there's a potential for overflow due to type confusion.

And I agree.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157331

Files:
  clang/docs/ReleaseNotes.rst
  clang/include/clang/Basic/DiagnosticSemaKinds.td
  clang/lib/Headers/CMakeLists.txt
  clang/lib/Headers/stdckdint.h
  clang/lib/Lex/ModuleMap.cpp
  clang/lib/Lex/PPDirectives.cpp
  clang/lib/Sema/SemaChecking.cpp
  clang/test/C/C2x/n2359.c
  clang/test/C/C2x/n2683.c
  clang/test/C/C2x/n2683_2.c
  clang/test/Headers/stdckdint.c
  clang/test/Modules/Inputs/System/usr/include/module.map
  clang/test/Sema/builtins-overflow.c
  clang/www/c_status.html

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D157331.554784.patch
Type: text/x-patch
Size: 19143 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230830/cd90c925/attachment-0001.bin>


More information about the cfe-commits mailing list