kbobyrev accepted this revision.
kbobyrev added a comment.
This revision is now accepted and ready to land.

Thanks for taking care of this!



================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:54
+
+- Diagnostics and fixes for unused include directives, according to IWYU style.
+  Off by default, can be turned on through `Diagnostics.IncludeCleaner
----------------
nit: probably link to the design doc?


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:63
+
+- New semantich hihlighting kinds for:
+  - Virtual methods
----------------



================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:73
+
+- Hover for include directives displays resolved paths.
+
----------------



================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:75
+
+- Include AKA types in hover, controlled with `Hover.ShowAKA
+  <https://clangd.llvm.org/config#showaka>`_ config option.
----------------
nit: desugared types? deduced types? AKA types is something I've never seen 
before.


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:78
+
+- Hover supports character literals.
+
----------------
nit: might make sense to move closer to the previous hover improvement for 
coherence


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118592

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

Reply via email to