[PATCH] D104192: [clang][RISCV] Change implicit ARCH for explicitly specified ABI

2021-06-13 Thread Ben Shi via Phabricator via cfe-commits
benshi001 added a comment. In D104192#2815757 , @MaskRay wrote: > The thing looks weird. My mental model (ppc) is that `-march` can infer > `-mabi`, not the other way around... The original way in Driver/Toolchains/Arch/RISCV.cpp is if (MABI.equals_l

[PATCH] D104192: [clang][RISCV] Change implicit ARCH for explicitly specified ABI

2021-06-13 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. The thing looks weird. My mental model is that `-march` can infer `-mabi`, not the other way around... Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104192/new/ https://reviews.llvm.org/D104192 ___

[PATCH] D104192: [clang][RISCV] Change implicit ARCH for explicitly specified ABI

2021-06-12 Thread Ben Shi via Phabricator via cfe-commits
benshi001 created this revision. benshi001 added reviewers: asb, luismarques, MaskRay, craig.topper. Herald added subscribers: vkmr, frasercrmck, evandro, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27