Bug 820751 - F17 gcc miscompiles lilypond
Summary: F17 gcc miscompiles lilypond
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: gcc
Version: 17
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Jakub Jelinek
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard: AcceptedNTH
Depends On:
Blocks: F17-accepted, F17FinalFreezeExcept
TreeView+ depends on / blocked
 
Reported: 2012-05-10 21:13 UTC by Jakub Jelinek
Modified: 2012-05-18 23:06 UTC (History)
6 users (show)

Fixed In Version: gcc-4.7.0-5.fc17
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-05-18 23:06:34 UTC
Type: Bug


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
GNU Compiler Collection 53239 0 None None None Never

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


Note You need to log in before you can comment on or make changes to this bug.