Bug 820751

Summary: F17 gcc miscompiles lilypond
Product: [Fedora] Fedora Reporter: Jakub Jelinek <jakub>
Component: gccAssignee: Jakub Jelinek <jakub>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 17CC: awilliam, bruno, dennis, jakub, law, tflink
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: AcceptedNTH
Fixed In Version: gcc-4.7.0-5.fc17 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-05-18 23:06:34 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 752653    

Description Jakub Jelinek 2012-05-10 21:13:23 UTC
See http://gcc.gnu.org/PR53239
It would be nice not to have this wrong-code bug in F17 GA, internal compiler errors etc. are less severe than silent miscompilations.

https://admin.fedoraproject.org/updates/FEDORA-2012-7449/gcc-4.7.0-5.fc17,gcc-python-plugin-0.9-4.fc17
fixes this bug.

Comment 1 Jakub Jelinek 2012-05-10 21:15:22 UTC
Additionally, gcc-python-plugin will not install in current F17 tree, as it hasn't been rebuilt against any recent gcc.

Comment 2 Adam Williamson 2012-05-10 22:38:49 UTC
I think this is meant to be proposed as NTH, not blocker.



-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

Comment 3 Dennis Gilmore 2012-05-12 02:20:03 UTC
+1 NTH, I can see people doinga f17 install, and never updating it then wondering whats wrong with gcc.

Comment 4 Bruno Wolff III 2012-05-13 13:22:40 UTC
-1 NTH this can be fixed with an update and it's impact on live images is going to be pretty small. (As far as I can tell gcc-python-plugin shouldn't end up on the install or live images; if it does and there is a problem installing that version, then I think this would be a blocker.)

Comment 5 Tim Flink 2012-05-14 17:51:16 UTC
-1 NTH as this could be fixed with an update and AFAIK, isn't on the DVD so users would have to hit the network to install this anyways.

Comment 6 Adam Williamson 2012-05-14 23:13:39 UTC
Tim: gcc is on the DVD.



-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

Comment 7 Tim Flink 2012-05-14 23:44:38 UTC
(In reply to comment #6)
> Tim: gcc is on the DVD.

Yes, I know that but I thought this was an issue with gcc-python-plugin which, as far as I know, isn't on the DVD.

If this is indeed an issue with the base gcc and not just gcc-python[23]-plugin, then I am less -1 NTH on this

I don't pretend to completely understand the upstream bug but what are the possible side effects?

Comment 8 Jakub Jelinek 2012-05-15 05:37:46 UTC
There is a gcc-c++ bug (well, several, this one is just most severe), where e.g.
(for complete testcase look up the upstream bug):

M
test (M *x)
{
  M n (R (0, 0));

  for (int i = 0; i < 2; i++)
  {
    M p = x[i];
    n = min (n, p);
  }
 	
  return n;
}
from the testcase is miscompiled, can affect most of the C++ code (because named return value optimization is very common in C++).  Here because M is not a POD type, the value is returned to memory pointed by a hidden argument, and NRV optimizes the code to have the variable n constructed in that memory.  What the bug is about is that the value range propagation pass computed badly the expected range of the hidden argument (assumed it is uninitialized), and if you are unlucky enough like lilypond was, it can result in silent miscompilation of it.
The gcc bug only affects gcc-* packages.

The second issue is just that gcc-python-plugin, which depends on gcc exact NVR, hasn't been rebuilt for quite a while, so it has broken dependencies.

Comment 9 Tim Flink 2012-05-18 21:22:05 UTC
Discussed in the 2012-05-18 blocker bug review meeting. Accepted as NTH for Fedora 17 final because this could cause problems with c++ compilation with the gcc packages released on DVDs.

Comment 10 Adam Williamson 2012-05-18 23:06:34 UTC
The update is pushed stable, so closing.



-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers