[clang] [Driver][AVR] Reject c/c++ compilation for avr1 devices (PR #111798)
David Spickett via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 11 06:19:38 PDT 2024
================
@@ -400,6 +400,14 @@ void AVRToolChain::AddClangSystemIncludeArgs(const ArgList &DriverArgs,
void AVRToolChain::addClangTargetOptions(
const llvm::opt::ArgList &DriverArgs, llvm::opt::ArgStringList &CC1Args,
Action::OffloadKind DeviceOffloadKind) const {
+ // Reject C/C++ compilation for avr1 devices.
+ const Driver &D = getDriver();
+ std::string CPU = getCPUName(D, DriverArgs, getTriple());
+ std::optional<StringRef> FamilyName = GetMCUFamilyName(CPU);
+ if (CPU == "avr1" || (FamilyName && FamilyName->compare("avr1") == 0))
+ D.Diag(diag::err_drv_opt_unsupported_input_type) << "avr1"
+ << "c/c++";
----------------
DavidSpickett wrote:
You could instead put `-mmcu=attiny11` as the value, then users know what part of the command triggered the issue, though they may not be aware that this whole category is the same. In practice I wonder if that would be more useful for most people.
Also what does gcc say in this situation?
https://github.com/llvm/llvm-project/pull/111798
More information about the cfe-commits
mailing list