[PATCH] D27860: [LLD] Remove assert from Relocations.h

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 16 15:23:33 PST 2016


This revision was automatically updated to reflect the committed changes.
Closed by commit rL290002: Fix compiler warning. (authored by zturner).

Changed prior to commit:
  https://reviews.llvm.org/D27860?vs=81792&id=81804#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D27860

Files:
  lld/trunk/ELF/Relocations.h


Index: lld/trunk/ELF/Relocations.h
===================================================================
--- lld/trunk/ELF/Relocations.h
+++ lld/trunk/ELF/Relocations.h
@@ -98,7 +98,7 @@
 // RelExpr's as a constant bit mask and test for membership with a
 // couple cheap bitwise operations.
 template <RelExpr... Exprs> bool isRelExprOneOf(RelExpr Expr) {
-  assert(0 <= Expr && Expr < 64 && "RelExpr is too large for 64-bit mask!");
+  assert(0 <= Expr && (int)Expr < 64 && "RelExpr is too large for 64-bit mask!");
   return (uint64_t(1) << Expr) & RelExprMaskBuilder<Exprs...>::build();
 }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27860.81804.patch
Type: text/x-patch
Size: 594 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161216/3c774d6e/attachment.bin>


More information about the llvm-commits mailing list