[PATCH] D139305: [clang][driver] Make option '-mmcu' as an alias of option '-mcpu'
Fangrui Song via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Dec 6 09:27:47 PST 2022
MaskRay requested changes to this revision.
MaskRay added a comment.
This revision now requires changes to proceed.
This makes `-mmcu=` available for non-AVR targets, which is not right.
Even for AVR, if this is a plain alias, is this really necessary? Usually having more than one aliases for one functionality just create confusion as users don't know which one is the canonical.
I think one choice is for AVR users to migrate `-mmcu=` (if any) to `-mcpu=`, if there are a significant number of uses. Making a project buildable with Clang usually requires porting efforts, and this request may not be unreasonable.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139305/new/
https://reviews.llvm.org/D139305
More information about the cfe-commits
mailing list