[llvm-dev] compiler-rt linux-arm builtins/clear_cache.c depends on kernel headers
Saleem Abdulrasool via llvm-dev
llvm-dev at lists.llvm.org
Sun Feb 12 15:18:05 PST 2017
Hmm, I dont think that this is terrible. However, since I dont belive that
Linux guarantees the stability of its interfaces, I think that adding a
trivial test for this would be waranteed.
On Sun, Feb 12, 2017 at 2:26 PM, Renato Golin <renato.golin at linaro.org>
> On 26 January 2017 at 10:57, Joakim Sindholt via llvm-dev
> <llvm-dev at lists.llvm.org> wrote:
> > It's difficult to ascertain whether it's mixing with libc headers
> > but if it is then that's a bug in its own right.
> > Either way, it's pulling in kernel headers for a single #defined
> > constant that's been part of the linux ABI for years now. This
> > makes it more difficult to cross compile for, in my opinion, no
> > good reason. So here's a patch to just place the #define inline.
> Hi Joakim,
> Sorry it took so long. I'm copying Saleem who knows more than I do about
> That define seems harmless. I checked kernel 3.13 and current trunk
> and the defines are the same. For compatibility, I seriously doubt
> they're going to change it anytime soon.
> I'd just add a comment where that line is from
> Saleem, any objections?
compnerd (at) compnerd (dot) org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-dev