From 30aa3bd42f381ad88edee4327460cf9bf25918a2 Mon Sep 17 00:00:00 2001
From: Daniel Grunwald <daniel@danielgrunwald.de>
Date: Tue, 3 Oct 2017 00:29:07 +0200
Subject: [PATCH] Fix missing cast in signed uint<int comparison.

---
 .../CSharp/ExpressionBuilder.cs                   | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/ICSharpCode.Decompiler/CSharp/ExpressionBuilder.cs b/ICSharpCode.Decompiler/CSharp/ExpressionBuilder.cs
index 2ecd43d1ab..4528bece8f 100644
--- a/ICSharpCode.Decompiler/CSharp/ExpressionBuilder.cs
+++ b/ICSharpCode.Decompiler/CSharp/ExpressionBuilder.cs
@@ -546,14 +546,15 @@ TranslatedExpression TranslateComp(Comp inst)
 			// attempt comparison without any additional casts
 			var rr = resolver.ResolveBinaryOperator(inst.Kind.ToBinaryOperatorType(), left.ResolveResult, right.ResolveResult)
 				as OperatorResolveResult;
-			if (rr != null && !rr.IsError
-				&& NullableType.GetUnderlyingType(rr.Operands[0].Type).GetSign() == inst.Sign
-				&& NullableType.GetUnderlyingType(rr.Operands[1].Type).GetSign() == inst.Sign)
-			{
-				return new BinaryOperatorExpression(left.Expression, op, right.Expression)
-					.WithILInstruction(inst)
-					.WithRR(rr);
+			if (rr != null && !rr.IsError) {
+				IType compUType = NullableType.GetUnderlyingType(rr.Operands[0].Type);
+				if (compUType.GetSign() == inst.Sign && compUType.GetStackType() == inst.InputType) {
+					return new BinaryOperatorExpression(left.Expression, op, right.Expression)
+						.WithILInstruction(inst)
+						.WithRR(rr);
+				}
 			}
+
 			// Ensure the inputs have the correct sign:
 			KnownTypeCode inputType = KnownTypeCode.None;
 			switch (inst.InputType) {