[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