为什么编译器会添加一个不必要的局部变量？

Question

这段C#代码：

private void LoadAssignments(AssignmentType assignmentType, Collection<Assignment> assignments)
    {
        bool flag;
        DataTable lessons = this.GetResults(assignmentType);
        try
        {
            IEnumerator enumerator = lessons.Rows.GetEnumerator();
            try
            {
                while (true)
                {
                    flag = enumerator.MoveNext();
                    if (!flag)
                    {
                        break;
                    }
                    DataRow row = (DataRow)enumerator.Current;
                }
            }
            finally
            {
                IDisposable disposable = enumerator as IDisposable;
                flag = disposable == null;
                if (!flag)
                {
                    disposable.Dispose();
                }
            }
        }
        finally
        {
            flag = lessons == null;
            if (!flag)
            {
                lessons.Dispose();
            }
        }
    }

生成这个CIL（.NET 4）。

    .method private hidebysig 
    instance void LoadAssignments (
        valuetype TTReporterCore.AssignmentType assignmentType,
        class [mscorlib]System.Collections.ObjectModel.Collection`1<valuetype TTReporterCore.Assignment> assignments
    ) cil managed 
{
    .locals init (
        [0] bool flag,
        [1] class [System.Data]System.Data.DataTable lessons,
        [2] class [mscorlib]System.Collections.IEnumerator enumerator,
        [3] class [System.Data]System.Data.DataRow row,
        [4] class [mscorlib]System.IDisposable disposable,
        [5] bool flag1
    )

    IL_0000: nop
    IL_0001: ldarg.0
    IL_0002: ldarg.1
    IL_0003: call instance class [System.Data]System.Data.DataTable TTReporterCore.TTReader::GetResults(valuetype TTReporterCore.AssignmentType)
    IL_0008: stloc.1
    .try
    {
        IL_0009: nop
        IL_000a: ldloc.1
        IL_000b: callvirt instance class [System.Data]System.Data.DataRowCollection [System.Data]System.Data.DataTable::get_Rows()
        IL_0010: callvirt instance class [mscorlib]System.Collections.IEnumerator [System.Data]System.Data.InternalDataCollectionBase::GetEnumerator()
        IL_0015: stloc.2
        .try
        {
            IL_0016: nop
            IL_0017: br.s IL_0038
            .loop
            {
                IL_0019: nop
                IL_001a: ldloc.2
                IL_001b: callvirt instance bool [mscorlib]System.Collections.IEnumerator::MoveNext()
                IL_0020: stloc.0
                IL_0021: ldloc.0
                IL_0022: stloc.s flag1
                IL_0024: ldloc.s flag1
                IL_0026: brtrue.s IL_002b

                IL_0028: nop
                IL_0029: br.s IL_003d

                IL_002b: ldloc.2
                IL_002c: callvirt instance object [mscorlib]System.Collections.IEnumerator::get_Current()
                IL_0031: castclass [System.Data]System.Data.DataRow
                IL_0036: stloc.3
                IL_0037: nop

                IL_0038: ldc.i4.1
                IL_0039: stloc.s flag1
                IL_003b: br.s IL_0019
            }

            IL_003d: nop
            IL_003e: leave.s IL_0062
        }
        finally
        {
            IL_0040: nop
            IL_0041: ldloc.2
            IL_0042: isinst [mscorlib]System.IDisposable
            IL_0047: stloc.s disposable
            IL_0049: ldloc.s disposable
            IL_004b: ldnull
            IL_004c: ceq
            IL_004e: stloc.0
            IL_004f: ldloc.0
            IL_0050: stloc.s flag1
            IL_0052: ldloc.s flag1
            IL_0054: brtrue.s IL_0060

            IL_0056: nop
            IL_0057: ldloc.s disposable
            IL_0059: callvirt instance void [mscorlib]System.IDisposable::Dispose()
            IL_005e: nop
            IL_005f: nop

            IL_0060: nop
            IL_0061: endfinally
        }

        IL_0062: nop
        IL_0063: nop
        IL_0064: leave.s IL_007e
    }
    finally
    {
        IL_0066: nop
        IL_0067: ldloc.1
        IL_0068: ldnull
        IL_0069: ceq
        IL_006b: stloc.0
        IL_006c: ldloc.0
        IL_006d: stloc.s flag1
        IL_006f: ldloc.s flag1
        IL_0071: brtrue.s IL_007c

        IL_0073: nop
        IL_0074: ldloc.1
        IL_0075: callvirt instance void [System]System.ComponentModel.MarshalByValueComponent::Dispose()
        IL_007a: nop
        IL_007b: nop

        IL_007c: nop
        IL_007d: endfinally
    }

    IL_007e: nop
    IL_007f: ret
}

为什么MSIL添加flag1标记，继续执行相同逻辑以设置标记，将flag1设置为flag，最后检查!flag1。这对我来说似乎是编译器效率低下。

更新：我使用的是Telerik的JustDecompile，虽然结果与ILDASM非常不同，但在调试模式下仍会创建附加布尔值。

此外，我通过完全删除布尔值修改了代码，调试版本仍会添加布尔值。我真的想知道编译器为什么要这样做。

- kakridge

---

如果您在发布模式下编译而不是调试模式下，会发生什么？ - Michael Liu

实际变量名类似于“CS$4$0000”，而不是“flag1”。在发布版本中，它确实被优化掉了。不确定您使用的反汇编器是什么，但听起来有问题。请使用ildasm.exe查看此内容。 - Hans Passant

Hans和Michael - 你们说得对。当我反汇编发布版本时，我可能弄错了。但是，并不是发布版本被优化，而是调试版本添加了一些东西进去。那么，第二个布尔变量是用于调试目的的吗？还是这是在调试标志被设置时编译器中的一个错误？我认为并不需要第二个布尔变量。 - kakridge

没有任何东西，这就是为什么它被优化掉了。在调试版本中也没有生成临时文件的理由，但编译器就是这样工作的，如果代码看起来不同，可能会需要它。它们有点愚蠢，除非你要求它们更努力。而你可以使用 /optimize 选项来实现这一点。 - Hans Passant

Answer 1

看起来会创建一个临时变量来保存比较的结果（即disposable == null）。

尝试运行这个例子：

class Program
{
   static void Main()
   {
       if (1 == 1) return;
   }
}

在我的电脑上，..产生了以下IL代码（Microsoft (R) Visual C# 2010编译器版本4.0.30319.1）：

.method private hidebysig static void  Main() cil managed
{
  .entrypoint
  .maxstack  1
  .locals init (bool V_0)
  IL_0000:  nop
  IL_0001:  ldc.i4.0
  IL_0002:  stloc.0
  IL_0003:  br.s       IL_0005
  IL_0005:  ret
}

即使实际上没有使用，也会创建本地变量 V_0。我相信这是一个明显的优化，即使在非优化编译中:) 或者更可能的是：生成了所有必要的代码以启用调试。我不知道它在调试会话中可能如何使用，但这是我最好的猜测。

当编译器进行优化（即发布配置）时，我看不到额外的局部变量。

- ziya

Answer 2

当我编写类似的程序时，额外的变量在发布模式下没有出现。在汇编视图中也是如此，调试显示额外的变量被设置为([ebp-44h])，而发布模式则没有： 调试：

发布：

- Jason Goemaat

---

你似乎跳过了标题的第一个单词。 - Kendall Frey