[llvm-commits] [llvm-gcc] [PATCH] psp allegrex support
Bruno Cardoso Lopes
bruno.cardoso at gmail.com
Sat Jun 28 00:07:56 PDT 2008
Hi Eric,
Nice to have your point of view.
I got everything from the homebrew toolchain, but your way seems
cleaner, i'll get it done. :)
>> <allegrex.patch>
>
> +mipsallegrex-*-elf* | mipsallegrexel-*-elf*)
>
> Why not just mipsallegrex*?
>
> + tm_defines="MIPS_ISA_DEFAULT=2 MIPS_CPU_STRING_DEFAULT=\\\"allegrex\\
> \" MIPS_ABI_DEFAULT=ABI_EABI"
>
> you want:
>
> tm_defines="${tm_defines} ...."
>
> - mips-*-*) echo mips;; \
> - mipsel-*-*) echo mipsel;; \
> + mips*el-*-*) echo mipsel;; \
> + mips*-*-*) echo mips;; \
>
> If this change is needed then something is wrong.
>
> +++ gcc/config/mips/t-allegrex (revision 0)
>
> You could just require --disable-multilibs and use t-elf, but that's
> not a big deal.
>
> +#undef SUBTARGET_CPP_SPEC
> +#define SUBTARGET_CPP_SPEC "-DPSP=1 -D__psp__=1 -D_PSP=1"
>
> This needs to go into TARGET_CPU_CPP_BUILTINS().
>
> You also need to define a PROCESSOR_TYPE and mask for the processor.
> That will make any other defines you need in the backend much easier
> to deal with (and enable you to do the above bits).
--
Bruno Cardoso Lopes
http://www.brunocardoso.cc
"When faced with untenable alternatives, you
should consider your imperative."
More information about the llvm-commits
mailing list