On 29 December 2013 23:53, Alp Toker <[email protected]> wrote: > Hi Rafael, > > This is all that's left to enable opportunistic checking at clang IRGen time > where an LLVM TargetMachine is available. > > include/clang/CodeGen/BackendUtil.h | 4 ++-- > lib/CodeGen/BackendUtil.cpp | 11 ++++++++--- > lib/CodeGen/CodeGenAction.cpp | 3 ++- > 3 files changed, 12 insertions(+), 6 deletions(-) > > Re-posting the updated patch as I lost track of the original thread, and > CC'ing in llvm-commits. > > You mentioned there's a remaining inconsistent ARM DataLayout that needs to > get fixed before we switch this on -- was it determined if it's the LLVM or > clang string that's at fault?
Not so much at fault. The issue is that llvm uses only the triple, where clangs uses an attribute for the abi. I will ping that patch and CC you. Sorry for disappearing. I am spending the holidays with my parents, so reading just a bit of email. Cheers, Rafael _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
