This is the mail archive of the gcc-patches@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]

Re: i386 attribute memory bug



  In message <19981027152710.A1565@dot.cygnus.com>you write:
  > I discovered the following debugging my flow reorg:
  > 
  > The calculation of attribute memory is not safe.  When applied
  > to return_internal, we get a segv.
  > 
  > This is because instruction types are defaulted to "integer" 
  > which assumes two input operands.  Return_internal does not,
  > and so the system crashes.  This comes up when sched2 tries
  > to determine the priority of the return.
  > 
  > It is likely that this has not been seen before because basic
  > block boundaries were not correct wrt the prologue and epilogue.
  > 
  > The following is the most conservative patch.  Instead of 
  > defaulting instructions to "integer", default them to "other".
How about fixing return_internal instead to specify "other" instead of
changing the default for everything.


Our scheduling is bad enough, this (I suspect) would make it much worse.

jeff


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