[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