------- Additional Comments From mark at codesourcery dot com 2005-05-03 19:35 ------- Subject: Re: [4.0 Regression] Reload may generate stores to read-only memory
Jeffrey A Law wrote: > On Sat, 2005-04-30 at 22:10 +0000, ebotcazou at gcc dot gnu dot org > wrote: > >>------- Additional Comments From ebotcazou at gcc dot gnu dot org 2005-04-30 >>22:10 ------- >>The same fix was applied to mainline. Jeff, please consider backporting your >>patch to 4.0 branch so that we be definitely done with this nasty problem. > > No strong opinions here -- it's certainly a latent bug, but triggering > it is relatively difficult with 4.0. > > Basically we need to evaluate the risk/reward. We can fix this with the > exact same patch that we're using in mainline right now which hasn't > caused us any problems yet. That patch is also very similar to what was > done for one or more of the 3.x release branches. I consider the patch > relatively safe. > > What's the reward -- a latent code generation bug is fixed. > > So is the reward worth the small risk the patch poses? > > [ Note that if we had a bug report with 4.0 which was traceable to > this problem, then I'd clearly land on the side of installing the > patch. ] I don't know that I'm in a position to make a good decision here, but I do agree with Jeff -- it would be a lot more compelling to make the change if there was actually a demonstrable problem -- especially coming from non-synthetic code. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=15248
