[llvm] 49688b3 - Fix `-Wpedantic` warning. NFC.

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 1 16:21:16 PDT 2020


Best if the patch description (especially the first line) describes
the change, rather than (or in addition to) the warning its
addressing. eg: "Remove unnecessary semicolon (-Wextra-semi)" or the
like

On Wed, May 27, 2020 at 12:57 PM Michael Liao via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
>
>
> Author: Michael Liao
> Date: 2020-05-27T15:57:03-04:00
> New Revision: 49688b3c306d0bf918c0abeee030cfd56a17c348
>
> URL: https://github.com/llvm/llvm-project/commit/49688b3c306d0bf918c0abeee030cfd56a17c348
> DIFF: https://github.com/llvm/llvm-project/commit/49688b3c306d0bf918c0abeee030cfd56a17c348.diff
>
> LOG: Fix `-Wpedantic` warning. NFC.
>
> Added:
>
>
> Modified:
>     llvm/tools/llvm-cov/CoverageFilters.h
>
> Removed:
>
>
>
> ################################################################################
> diff  --git a/llvm/tools/llvm-cov/CoverageFilters.h b/llvm/tools/llvm-cov/CoverageFilters.h
> index ccaa7a9df590..33fd9929c59a 100644
> --- a/llvm/tools/llvm-cov/CoverageFilters.h
> +++ b/llvm/tools/llvm-cov/CoverageFilters.h
> @@ -23,7 +23,7 @@ class SpecialCaseList;
>  namespace coverage {
>  class CoverageMapping;
>  struct FunctionRecord;
> -}; // namespace coverage
> +} // namespace coverage
>
>  /// Matches specific functions that pass the requirement of this filter.
>  class CoverageFilter {
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits


More information about the llvm-commits mailing list