[llvm-commits] [llvm-gcc-4.2] r55650 - /llvm-gcc-4.2/trunk/gcc/llvm-convert.cpp

[Duncan Sands]

> > llvm-convert.cpp is target independent code.
> 
> No, it's not; there's lots of #ifdef TARGET_LLVM's in here.  (If  
> you're saying it's not target-dependent because it's using some  
> abstraction rather than TARGET_POWERPC, I strongly disagree.)

Well, there aren't that many!  Most of them are for sync things
and were introduced by you ;)  Otherwise there is this one:

 49950  evancheng #if defined(TARGET_ALPHA) || defined(TARGET_386) || defined(TARGET_POWERPC)

And that's it really.

Teasing aside, I would much rather that LLVM intrinsics were
always output, and that they get turned into library calls by
the LLVM back-end if not supported on the platform.  That
makes things uniform for all front-ends, rather than each one
having to know which intrinsics are supported and which not.

Ciao,

Duncan.