> 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.)
Exactly, the issue is the warning. GCC handles this as a fully magic function, no declaration in a header is needed. >> 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. Sorry, would you mind reviewing the new version attached? Trying to bootstrap clang I noticed that the name was being mangle in c++ :-( If you think the assert should stay, I can also make GetOrCreateLLVMFunction public. > -Eli > Thanks, -- Rafael Ávila de Espíndola
clear_cache.patch
Description: Binary data
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
