[PATCH] D118333: [RISCV] Update computeTargetABI implementation.

2022-01-27 Thread Zakk Chen via Phabricator via cfe-commits
khchen added a comment. In D118333#3275940 , @jrtc27 wrote: > I think this is the same idea as D118333 ? > Other than being a cleaner way of achieving the same goal. I've not looked to > see if there are any functional

[PATCH] D118333: [RISCV] Update computeTargetABI implementation.

2022-01-27 Thread Jessica Clarke via Phabricator via cfe-commits
jrtc27 added a comment. Uh D113959 even Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118333/new/ https://reviews.llvm.org/D118333 ___ cfe-commits mailing list cfe-comm

[PATCH] D118333: [RISCV] Update computeTargetABI implementation.

2022-01-27 Thread Jessica Clarke via Phabricator via cfe-commits
jrtc27 added a comment. I think this is the same idea as D118333 ? Other than being a cleaner way of achieving the same goal. I've not looked to see if there are any functional differences between the two. Repository: rG LLVM Github Monorepo CHANGES SINCE L

[PATCH] D118333: [RISCV] Update computeTargetABI implementation.

2022-01-27 Thread Zakk Chen via Phabricator via cfe-commits
khchen created this revision. khchen added reviewers: craig.topper, jrtc27, frasercrmck, kito-cheng, arcbbb, monkchiang, eopXD. Herald added subscribers: VincentWu, luke957, achieveartificialintelligence, armkevincheng, eric-k256, vkmr, dexonsmith, evandro, luismarques, apazos, sameer.abuasal, s