[PATCH] D98253: [clang][ARM] Refactor ComputeLLVMTriple code for ARM

2021-03-11 Thread David Spickett via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG87358dba8182: [clang][ARM] Refactor ComputeLLVMTriple code for ARM (authored by DavidSpickett). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98253/new/

[PATCH] D98253: [clang][ARM] Refactor ComputeLLVMTriple code for ARM

2021-03-10 Thread Oliver Stannard (Linaro) via Phabricator via cfe-commits
ostannard accepted this revision. ostannard added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98253/new/ https://reviews.llvm.org/D98253

[PATCH] D98253: [clang][ARM] Refactor computeLLVMTriple code for ARM

2021-03-09 Thread David Spickett via Phabricator via cfe-commits
DavidSpickett added a reviewer: ostannard. DavidSpickett added a comment. Besides the usual motivations this is prep for fixing https://bugs.llvm.org/show_bug.cgi?id=48894 for AArch64. Which will need a similar (but simpler) set of calls. Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D98253: [clang][ARM] Refactor computeLLVMTriple code for ARM

2021-03-09 Thread David Spickett via Phabricator via cfe-commits
DavidSpickett created this revision. Herald added subscribers: danielkiss, kristof.beyls. DavidSpickett requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This moves code that sets the architecture name and Float ABI into two new functions in