Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline

Differential D142592 Diff 492597 clang-tools-extra/docs/ReleaseNotes.rst

Changeset View

Standalone View

View Options

clang-tools-extra/docs/ReleaseNotes.rst

Show First 20 Lines • Show All 91 Lines • ▼ Show 20 Lines
The improvements are... The improvements are...
Improvements to clang-tidy Improvements to clang-tidy
-------------------------- --------------------------
New checks New checks
^^^^^^^^^^ ^^^^^^^^^^
New `llvmlibc-inline-function-decl-check <http://clang.llvm.org/extra/clang-tidy/checks/llvmlibc/inline-function-decl-check.html>`_
carlosgalvezpUnsubmitted
Done
ReplyInline Actions

I believe you'll need a rebase since a new check was recently added.

carlosgalvezp: I believe you'll need a rebase since a new check was recently added.
carlosgalvezpUnsubmitted
Done
ReplyInline Actions

Remove

carlosgalvezp: Remove
carlosgalvezpUnsubmitted
Done
ReplyInline Actions

Remove

carlosgalvezp: Remove
check.
Checks that all implicit and explicit inline functions in header files are
tagged with the ``LIBC_INLINE`` macro. See the `libc style guide
Eugene.ZelenkoUnsubmitted
Done
ReplyInline Actions

One sentence is enough.

Eugene.Zelenko: One sentence is enough.
carlosgalvezpUnsubmitted
Done
ReplyInline Actions

You need to also add the check to clang-tools-extra/docs/clang-tidy/checks/list.rst

carlosgalvezp: You need to also add the check to `clang-tools-extra/docs/clang-tidy/checks/list.rst`
<https://libc.llvm.org/code_style.html>`_ for more information about this macro.
New check aliases New check aliases
^^^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^
Changes in existing checks Changes in existing checks
^^^^^^^^^^^^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^^^^^^
Removed checks Removed checks
^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^
Show All 21 Lines