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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to