Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/arch



On Fri, Feb 23, 2018 at 01:12:13PM +0100, Maxime Villard wrote:
> > Heck, it wasn't even discussed on tech-kern.
> 
> It was discussed on tech-kern.

No, you made some wild claims about GDB behavior which are wrong and
without context. That's not a discussion about a code change.

> > If anything, this should be done in the kernel config next to the
> > -fno-omit-frame-potiner setting. This is a concession to DDB only,
> > unlike i.e. -mcmode=kernel, it is not necessary for ABI reasons or
> > correctness,
> 
> Moving -fno-shrink-wrap next to -fno-omit-frame-pointer did not fix the
> issues. That is to say, the frame was _not_ pushed at the beginning of the
> functions.

Then figure out why not. Placing random pessimisation options all over
the place is wrong.

Joerg


Home | Main Index | Thread Index | Old Index