[llvm] eace2fb - Fix warning on trailing `;`. NFC.

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 26 11:38:43 PDT 2020


It's handy (at least to me) to describe how you're fixing a warning, rather
than only that you are fixing it - "remove unnecessary semicolon" I think
would be a more descriptive... description? (could include that this was
found by a warning, or not - that seems less important than the
description/motivation underlying the warning)

On Tue, Feb 4, 2020 at 5:42 PM Michael Liao via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

>
> Author: Michael Liao
> Date: 2020-02-04T20:42:05-05:00
> New Revision: eace2fbfee6675fb9d85290b7d1d4d10c5d98f5d
>
> URL:
> https://github.com/llvm/llvm-project/commit/eace2fbfee6675fb9d85290b7d1d4d10c5d98f5d
> DIFF:
> https://github.com/llvm/llvm-project/commit/eace2fbfee6675fb9d85290b7d1d4d10c5d98f5d.diff
>
> LOG: Fix warning on trailing `;`. NFC.
>
> Added:
>
>
> Modified:
>     llvm/include/llvm/IR/LLVMContext.h
>
> Removed:
>
>
>
>
> ################################################################################
> diff  --git a/llvm/include/llvm/IR/LLVMContext.h
> b/llvm/include/llvm/IR/LLVMContext.h
> index e93b1a4e517e..a6dbf2b671c5 100644
> --- a/llvm/include/llvm/IR/LLVMContext.h
> +++ b/llvm/include/llvm/IR/LLVMContext.h
> @@ -39,7 +39,7 @@ class raw_ostream;
>
>  namespace remarks {
>  class RemarkStreamer;
> -};
> +}
>
>  namespace SyncScope {
>
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200326/8201eacf/attachment.html>


More information about the llvm-commits mailing list