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

Galina Kistanova via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 6 23:53:39 PDT 2017


r304874.

On Tue, Jun 6, 2017 at 11:48 PM, Craig Topper <craig.topper at gmail.com>
wrote:

> 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/Symbo
>> lTable.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/b9e60ccb/attachment.html>


More information about the llvm-commits mailing list