This is the mail archive of the gcc@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PR 8134: C++ crash


Mark Mitchell wrote:
PR 8134 is a crash in force_store_init_value on the branch; it is a
regression.

a) Nothing
b) Move my changes over.
c) Revert Alexandre's patch.
None of these are ideal are they :) Was Alexandre's patch fixing a
regression? If it was, then (c) is not really better than (a), either
way we reintroduce a regression. Though (c) has the slight advantage
of not changing the regression.

If the patch is not fixing a regression, then (c) is the best of the
bunch. Mark, you're the one who really knows how big (b) is, but as
an observer, it seems rather large and risky for a dot release.

nathan

--
Dr Nathan Sidwell   ::   http://www.codesourcery.com   ::   CodeSourcery LLC
         'But that's a lie.' - 'Yes it is. What's your point?'
nathan@codesourcery.com : http://www.cs.bris.ac.uk/~nathan/ : nathan@acm.org



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]