[llvm-commits] [llvm] r77903 - in /llvm/trunk: include/llvm/Intrinsics.td include/llvm/IntrinsicsBlackfin.td lib/Target/Blackfin/BlackfinInstrInfo.td test/CodeGen/Blackfin/load-intr.ll test/CodeGen/Blackfin/sync-intr.ll

[Jakob Stoklund Olesen]

On 02/08/2009, at 21.56, Eli Friedman wrote:

> On Sun, Aug 2, 2009 at 12:49 PM, Jakob Stoklund  
> Olesen<stoklund at 2pi.dk> wrote:
>> I agree. The only reason to add these builtins is that GCC has them.
>> If that is not reason enough, I will remove them. Should I?
>
> Well, you can both have them in gcc and translate them into IR; the
> pointer to llvm-i386.cpp was to point out how this is done.

I see what you mean. Let the front-end convert them to plain IR. The  
back-end doesn't have to know about them.

I will remove them from the back-end.

The blackfin backend that comes with llvm-gcc seems to be somewhat  
older than the one at http://blackfin.uclinux.org/ . I have never had  
the pleasure of working with GCC code before; I am not sure I have the  
courage to start, so I will probably not make a bfin-llvm-gcc frontend.

Clang support seems to be easy, though.

/jakob