Блог пользователя zero4338

Автор zero4338, история, 2 года назад, По-английски

The code below outputs -8 in cf custom test when using any GNU compiler.
It should output 2 and works well on other online IDEs and my computer.
It also works well on cf when using Clang.
What's the reason for this and how to avoid it?

#include<bits/stdc++.h>
using namespace std;
vector<pair<int,int>>f;
int mg(pair<int,int>x,pair<int,int>y)
{
	return max(x.second,y.second);
}
int main()
{
	f={{0,-11}};
	for(int J=0;J<=0;J++)
	{
		f[J]=f[0];
		if(J==0)f[J]={0,2};
 		cout<<mg({0,-8},f[J])<<endl;
	}
	return 0;
}
  • Проголосовать: нравится
  • +109
  • Проголосовать: не нравится

»
2 года назад, # |
  Проголосовать: нравится +16 Проголосовать: не нравится

upd: It also outputs -8 on my computer when using -O2.

»
2 года назад, # |
  Проголосовать: нравится +46 Проголосовать: не нравится

I tried to compile in different gcc versions with -O2 and it outputs -8. If the code doesn't have any issues, you should file a bug into GCC bug tracker.

»
2 года назад, # |
  Проголосовать: нравится +21 Проголосовать: не нравится

Seemed like -O2 change the execution order, you can add "volatile" before the variable to avoid it.

  • »
    »
    2 года назад, # ^ |
      Проголосовать: нравится +16 Проголосовать: не нравится
    Spoiler
  • »
    »
    2 года назад, # ^ |
      Проголосовать: нравится 0 Проголосовать: не нравится

    So when will this happen?
    It bothers to add volatile before all variables.

    • »
      »
      »
      2 года назад, # ^ |
      Rev. 3   Проголосовать: нравится +8 Проголосовать: не нравится

      No one knows. (idk at least)

      -O2 will change the execution order to reduce the data relativity. But the situation it happened or whether it is right is almost unpredictable.

»
2 года назад, # |
Rev. 3   Проголосовать: нравится +49 Проголосовать: не нравится

On Codeforces, merely changing the line if(J==0)f[J]={0,2}; to if (J==0) {f[J]={0,2}; printf("HERE");} causes the final output to turn into 2 from -8 (in addition to the added "HERE"). Same with adding the line printf("%d\n", J); before the if statement. It seems that any attempt to force the compiler to acknowledge what's going on changes the result.

This is very strong evidence of either a compiler bug or a strange instance of undefined behavior somewhere in the code. I don't see any reason for undefined behavior here though.

By the way, zero4338, how did you come up with this piece of code? It seems very artificially constructed.

»
2 года назад, # |
Rev. 2   Проголосовать: нравится 0 Проголосовать: не нравится

Adding either of #define _GLIBCXX_ASSERTIONS or #define _GLIBCXX_DEBUG "fix" the issue. Don't know why.

»
2 года назад, # |
  Проголосовать: нравится 0 Проголосовать: не нравится

I changed int mg(pair<int,int>x,pair<int,int>y) to int mg(pair<int,int>x, const pair<int,int> &y) and it outputs 2 but I didn't know why.

»
2 года назад, # |
  Проголосовать: нравится +16 Проголосовать: не нравится

To give this topic some ending, it seems somebody from this topic or from the original uoj topic has reported the bug (and also a duplicate) yesterday and it is being processed.