Mikhail Glushenkov wrote:> I removed the '-relocation-model' bit from the default invocation > string for llc. To pass arguments to llc, use the new "-Wllc" > option.I'd say that this is the proper solution, even though it breaks backward compatibility on x86_64. But given that llvmc is still considered experimental, better do it now than later. ;-) Thanks a lot for creating llvmc, it's very useful! -- Dr. Albert Gr"af Dept. of Music-Informatics, University of Mainz, Germany Email: Dr.Graef at t-online.de, ag at muwiinfa.geschichte.uni-mainz.de WWW: http://www.musikinformatik.uni-mainz.de/ag
Albert Graef <Dr.Graef <at> t-online.de> writes:> I'd say that this is the proper solution, even though it breaks backward > compatibility on x86_64. But given that llvmc is still considered > experimental, better do it now than later.A better solution probably would be to add '-relocation-model=pic' only when we are on x86_64. There is no easy way to access target information in plugins now, but I've filed a bug on it: http://llvm.org/bugs/show_bug.cgi?id=3920> Thanks a lot for creating llvmc, it's very useful!I'm glad to hear that:)
Mikhail Glushenkov wrote:> A better solution probably would be to add > '-relocation-model=pic' only when we are on x86_64.That still wouldn't work if you're cross-compiling to x86_32, since once the option is hardcoded there's no way to remove it from the command line. And it would be at odds with the rest of the LLVM toolchain which is pretty agnostic about the particular system it runs on. -- Dr. Albert Gr"af Dept. of Music-Informatics, University of Mainz, Germany Email: Dr.Graef at t-online.de, ag at muwiinfa.geschichte.uni-mainz.de WWW: http://www.musikinformatik.uni-mainz.de/ag