[PATCH] D73967: Implement _ExtInt as an extended int type specifier.
Melanie Blower via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 27 07:05:49 PST 2020
mibintc added a comment.
Added a couple inline comments
================
Comment at: clang/docs/ReleaseNotes.rst:62
+ permit non-power of 2 integers, exposing the LLVM integer types. Since a major
+ motivating use case for these types is to limit 'bit' useage, these types don't
+ automatically promote to 'int' when operations are done between two ``ExtInt(N)``
----------------
^useage^usage^ -- please also correct spelling in the code
================
Comment at: clang/include/clang/AST/ASTContext.h:1215
+ /// Return an extended integer type with the specified underlying type and bit
+ /// count.
----------------
Is this comment correct? "with the specified underlying type"
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D73967/new/
https://reviews.llvm.org/D73967
More information about the cfe-commits
mailing list