[llvm-branch-commits] [llvm-branch] r111406 - /llvm/branches/Apple/williamson/tools/lto/LTOModule.cpp

Daniel Dunbar daniel at zuster.org
Wed Aug 18 13:33:11 PDT 2010


Author: ddunbar
Date: Wed Aug 18 15:33:11 2010
New Revision: 111406

URL: http://llvm.org/viewvc/llvm-project?rev=111406&view=rev
Log:
Merge r110757:
--
Author: Daniel Dunbar <daniel at zuster.org>
Date:   Wed Aug 11 00:11:19 2010 +0000

    lto: Fix an inverted conditional which prevented the addition of symbols scraped
    from inline assembly, except in cases where they had already been seen (in which
    case they would get added twice).
     - I can't see how this ever worked...

Modified:
    llvm/branches/Apple/williamson/tools/lto/LTOModule.cpp

Modified: llvm/branches/Apple/williamson/tools/lto/LTOModule.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/branches/Apple/williamson/tools/lto/LTOModule.cpp?rev=111406&r1=111405&r2=111406&view=diff
==============================================================================
--- llvm/branches/Apple/williamson/tools/lto/LTOModule.cpp (original)
+++ llvm/branches/Apple/williamson/tools/lto/LTOModule.cpp Wed Aug 18 15:33:11 2010
@@ -356,7 +356,7 @@
 
 void LTOModule::addAsmGlobalSymbol(const char *name) {
   // only add new define if not already defined
-  if (_defines.count(name) == 0)
+  if (_defines.count(name))
     return;
 
   // string is owned by _defines





More information about the llvm-branch-commits mailing list