From dcced60532bd7df3973756599b06598d9d0d6b8c Mon Sep 17 00:00:00 2001 From: Simon Pilgrim Date: Mon, 29 Apr 2019 18:32:06 +0000 Subject: [PATCH] [X86] scaleShuffleMask - avoid potential signed overflow warning. Use size_t assignment to prevent a bad explicit type conversion warning. Given the typical size of shuffle masks this was never going to happen, but this at least stops the warning. Reported in https://www.viva64.com/en/b/0629/ git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@359479 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Target/X86/X86ISelLowering.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/Target/X86/X86ISelLowering.h b/lib/Target/X86/X86ISelLowering.h index 02f34173339..eee8466384d 100644 --- a/lib/Target/X86/X86ISelLowering.h +++ b/lib/Target/X86/X86ISelLowering.h @@ -1600,10 +1600,10 @@ namespace llvm { void scaleShuffleMask(int Scale, ArrayRef Mask, SmallVectorImpl &ScaledMask) { assert(0 < Scale && "Unexpected scaling factor"); - int NumElts = Mask.size(); - ScaledMask.assign(static_cast(NumElts * Scale), -1); + size_t NumElts = Mask.size(); + ScaledMask.assign(NumElts * Scale, -1); - for (int i = 0; i != NumElts; ++i) { + for (int i = 0; i != (int)NumElts; ++i) { int M = Mask[i]; // Repeat sentinel values in every mask element. -- 2.50.1