[PATCH] D55100: Fix build warnings introduced in rL347938

[Mircea Trofin via Phabricator via llvm-commits]

mtrofin created this revision.

Suppressed warnings in release builds due to variable used
only in assert statement.


Repository:
  rL LLVM

https://reviews.llvm.org/D55100

Files:
  lib/Target/X86/X86DiscriminateMemOps.cpp


Index: lib/Target/X86/X86DiscriminateMemOps.cpp
===================================================================
--- lib/Target/X86/X86DiscriminateMemOps.cpp
+++ lib/Target/X86/X86DiscriminateMemOps.cpp
@@ -114,7 +114,9 @@
         Changed = true;
         const std::pair<DenseSet<unsigned>::iterator, bool> MustInsert =
             Set.insert(DI->getBaseDiscriminator());
-        assert(MustInsert.second);
+        (void)MustInsert; // silence warning.
+        assert(MustInsert.second &&
+               "New discriminator shouldn't be present in set");
       }
 
       // Bump the reference DI to avoid cramming discriminators on line 0.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55100.176006.patch
Type: text/x-patch
Size: 651 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181130/92d41f43/attachment.bin>