g++ osx 上使用 -O3 的优化错误

Question

使用 -O3 编译时，以下代码会生成错误的退出代码。我认为内部循环被错误地优化掉了。使用 -O2 或 -fno-inline 它可以工作。生成一个更简单的示例很困难，因为任何小的更改和错误都会消失。

编译：

/usr/bin/g++ -O3 -o bugexample bugexample.cpp

代码：

#include <vector>

int test(std::vector<char>& a, int& b)
{
    std::vector<int> z;
    z.push_back(10);
    z.push_back(10);

    int d = (int)a.size();

    int x = 1;
    for (int j = 0; j < 2; j++)
    {
        int c = j - 1;

        for (int i = 0; i < d; i++) 
        {
            if (j == 0)
            {
            }
            else if (i == 0)
            {
            }
            else
            {
                if (a[j] == a[i - 1])
                {
                    b = c + 1;
                    x = 2;
                }
                z[i] = 1;
            }
        }
    }

    return x;
}


int main(int argc, char* argv[])
{
    std::vector<char> a;
    a.push_back('a');
    a.push_back('a');
    int b = 1;
    return test(a,b);
}

编译器版本：

/usr/bin/g++ -v
Using built-in specs.
Target: i686-apple-darwin10
Configured with: /var/tmp/gcc/gcc-5666.3~123/src/configure --disable-checking --enable-werror --prefix=/usr --mandir=/share/man --enable-languages=c,objc,c++,obj-c++ --program-transform-name=/^[cg][^.-]*$/s/$/-4.2/ --with-slibdir=/usr/lib --build=i686-apple-darwin10 --program-prefix=i686-apple-darwin10- --host=x86_64-apple-darwin10 --target=i686-apple-darwin10 --with-gxx-include-dir=/include/c++/4.2.1
Thread model: posix
gcc version 4.2.1 (Apple Inc. build 5666) (dot 3)

对任何 insite 感兴趣，或者证明这是我的错。

编辑：生成的退出代码是 1，而它应该是 2。

Answer 1

嗯，这是某种混淆代码竞赛吗？

据我所知，您正在尝试对输入向量进行某种回文测试。只是，

• 循环 var j 的硬编码上限为 2 （这可能也应该是 a.size() ？），
• 您只返回最后一个位置的检查
• 你有各种各样的冗余条件
• 你有无偿的非常量参数
• 你有未使用的z向量
• 你不必要地使用了int for bool（1 =＆gt; false - 未找到，2 =＆gt; true -发现）
• 你不必要地使用了输出参数b；我将 bool 返回类型替换为 b 的值（b==-1 表示未找到匹配项）

在简化这些内容的代码时，我得到了以下代码，并且（就像您自己的代码一样）它对于 g++ 4.6.1 上的所有优化级别的行为都是相同的：

#include <vector>

int test(const std::vector<char>& a)
{
    /* int j = 1; // was: for (int j = 1; j < 2; j++) */

    for (int i = a.size()-1; i > 1; i--) 
        if (a[1] == a[i - 1])
            return 1;

    return -1;
}


int main(int argc, char* argv[])
{
    std::vector<char> a(2, 'a');
    int b = test(a);

    return b==-1? 1 : 2;
}