[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