[lld] r304873 - Added LLVM_FALLTHROUGH to address warning: this statement may fall through. NFC.

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 6 23:48:46 PDT 2017


Delete the redundant comment?

~Craig

On Tue, Jun 6, 2017 at 11:46 PM, Galina Kistanova via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> Author: gkistanova
> Date: Wed Jun  7 01:46:38 2017
> New Revision: 304873
>
> URL: http://llvm.org/viewvc/llvm-project?rev=304873&view=rev
> Log:
> Added LLVM_FALLTHROUGH to address warning: this statement may fall
> through. NFC.
>
> Modified:
>     lld/trunk/lib/Core/SymbolTable.cpp
>
> Modified: lld/trunk/lib/Core/SymbolTable.cpp
> URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/Core/
> SymbolTable.cpp?rev=304873&r1=304872&r2=304873&view=diff
> ============================================================
> ==================
> --- lld/trunk/lib/Core/SymbolTable.cpp (original)
> +++ lld/trunk/lib/Core/SymbolTable.cpp Wed Jun  7 01:46:38 2017
> @@ -162,6 +162,7 @@ bool SymbolTable::addByName(const Atom &
>                     << existing->name() << " (" << existingSize << ") "
>                     << newAtom.name() << " (" << newSize << ")\n";
>        // fallthrough
> +      LLVM_FALLTHROUGH;
>      }
>      case MCR_Error:
>        llvm::errs() << "Duplicate symbols: "
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://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/20170606/3c7a0146/attachment.html>


More information about the llvm-commits mailing list