Fix ReduceNestingTransform

ICSharpCode.Decompiler/IL/ControlFlow/ConditionDetection.cs

@@ -347,7 +347,7 @@ namespace ICSharpCode.Decompiler.IL.ControlFlow
 		/// 
 		/// Assumes ifInst does not have an else block
 		/// </summary>
-		internal static void InvertIf(Block block, IfInstruction ifInst, ILTransformContext context)
+		internal static void InvertIf(Block block, IfInstruction ifInst, ILTransformContext context, bool forceBlock = true)
 		{
 			Debug.Assert(ifInst.Parent == block);
 			
@@ -362,11 +362,16 @@ namespace ICSharpCode.Decompiler.IL.ControlFlow
 			//save a copy
 			var thenInst = ifInst.TrueInst;
 
+			if (ifInst != block.Instructions.SecondToLastOrDefault() || forceBlock) {
 				// extract "else...; exit".
 				// Note that this will only extract instructions that were previously inlined from another block
 				// (via InlineExitBranch), so the instructions are already fully-transformed.
 				// So it's OK to move them into a nested block again (which hides them from the following block transforms).
 				ifInst.TrueInst = ExtractBlock(block, block.Instructions.IndexOf(ifInst) + 1, block.Instructions.Count);
+			} else {
+				block.Instructions.RemoveAt(block.Instructions.Count - 1);
+				ifInst.TrueInst = exitInst;
+			}
 
 			if (thenInst is Block thenBlock) {
 				block.Instructions.AddRange(thenBlock.Instructions);

ICSharpCode.Decompiler/IL/Transforms/ReduceNestingTransform.cs

@@ -213,7 +213,7 @@ namespace ICSharpCode.Decompiler.IL
 				
 				// use the same exit the block has. If the block already has one (such as a leave from a try), keep it in place
 				EnsureEndPointUnreachable(ifInst.TrueInst, block.Instructions.Last());
-				ConditionDetection.InvertIf(block, ifInst, context);
+				ConditionDetection.InvertIf(block, ifInst, context, forceBlock: false);
 
 				// ensure the exit inst of the if instruction is a keyword
 				Debug.Assert(!(ifInst.TrueInst is Block));