This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB 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: [commit] Adding support for Xtensa Call0 ABI


Joel Brobecker wrote:
Since adding new ABI support is a big patch I considered
that as a good opportunity to reformat comments
(capitals, spaces after periods, etc) and do some
minor clean-ups to match coding standard.

Actually, it's cleaner if you could separate the formatting-only
patch from the real change. The formatting changes just clutter
your patch, and make it harder for us to review your changes.
I was thinking clean-ups are pretty straight forward.
I got your point for the future and will separate patches
covering different issues. I'm sorry for any inconvenience.
Is this worth a NEWS entry?
Well, I don't feel like it is. In the near future, I plan to submit
Xtensa GNU Linux support introducing new target. That would be a
true addition to the NEWS and good opportunity to mention that FSF
Xtensa GDB supports both ABIs.


-- Maxim



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