This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 02/11] Initial TI PRU libgcc port
On 06/13/2018 12:57 PM, Dimitar Dimitrov wrote:
> The floating point support has been borrowed from C6X libgcc port
> to help with TI PRU toolchain ABI compatibility.
>
> libgcc/ChangeLog:
>
> 2018-06-13 Dimitar Dimitrov <dimitar@dinux.eu>
>
> * config.host: Add PRU target.
> * config/pru/asri.c: New file.
> * config/pru/eqd.c: New file.
> * config/pru/eqf.c: New file.
> * config/pru/ged.c: New file.
> * config/pru/gef.c: New file.
> * config/pru/gtd.c: New file.
> * config/pru/gtf.c: New file.
> * config/pru/led.c: New file.
> * config/pru/lef.c: New file.
> * config/pru/lib2bitcountHI.c: New file.
> * config/pru/lib2divHI.c: New file.
> * config/pru/lib2divQI.c: New file.
> * config/pru/lib2divSI.c: New file.
> * config/pru/libgcc-eabi.ver: New file.
> * config/pru/ltd.c: New file.
> * config/pru/ltf.c: New file.
> * config/pru/mpyll.S: New file.
> * config/pru/pru-abi.h: New file.
> * config/pru/pru-asm.h: New file.
> * config/pru/pru-divmod.h: New file.
> * config/pru/sfp-machine.h: New file.
> * config/pru/t-pru: New file.
I don't see anything particularly concerning here. It looks like the
floating point stuff is a direct copy from the c6x port. I'm going to
assume the assembly code you've written in the various files is correct.
I'm also assuming we're trying to be compatible with another toolchain
or published ABI, hence the exposure of the new symbols for low level
operations.
So I'll this too. Obviously it needs to wait for the main port patch to
be reviewed as well.
jeff