Eugene.Zelenko added inline comments.

================
Comment at: clang/docs/ReleaseNotes.rst:70
   motivating use case for these types is to limit 'bit' usage, these types 
don't
-  automatically promote to 'int' when operations are done between two 
``ExtInt(N)``
-  types, instead math occurs at the size of the largest ``ExtInt(N)`` type.
+  automatically promote to 'int' when operations are done between two
+  ``ExtInt(N)`` types, instead math occurs at the size of the largest
----------------
I think //int// should be highlighted too with double back-ticks.


================
Comment at: clang/docs/ReleaseNotes.rst:114
+  this occurs when the use of the ``extern`` keyword is neglected in the
+  declaration
   of a variable in a header file. In some cases, no specific translation unit
----------------
MyDeveloperDay wrote:
> Eugene.Zelenko wrote:
> > This and next line should be merged.
> I knew you'd be perfect to review this...
I'm happy to help!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80031

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

Reply via email to