Source-Changes-D archive

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

Re: CVS commit: src/sys/arch



Le 23/02/2018 à 15:07, Maxime Villard a écrit :
Le 23/02/2018 à 14:55, Joerg Sonnenberger a écrit :
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.

Pure idiocy, go back read the thread.

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.

And also, could you expand a little bit about what is inherently wrong with
putting -fno-shrink-wrap on CFLAGS and not DEFCOPTS?

Maxime


Home | Main Index | Thread Index | Old Index