On Fri, Jun 4, 2010 at 7:10 PM, Rafael Espindola <[email protected]> wrote:
> The attached patch implements __clear_cache for the ARM. The
> implementation is not as simple as I would expect. The problem is that
> I cannot use "f" because there is no header defining it and cannot use
> "F" because it doesn't start with __builtin (we would have a call to
> "che" :-) ).

I think you can use "f" without specifying a header name, but I'm
guessing you don't want the warning asking for an explicit declaration
of __clear_cache?  (I don't have an ARM gcc around to check how it
behaves.)

> The patch also removes some dead code. There is not need to special
> case ARM::BI__builtin_thread_pointer since the generic intrinsic
> handling code takes care of it.

Patch looks fine.

-Eli
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to