Hi Dylan, I reverted this in r294180 because I think it broke some of the buildbots. See for example http://lab.llvm.org:8011/builders/clang-cmake-armv7-a15/builds/3599 http://lab.llvm.org:8011/builders/clang-x86-windows-msvc2015/builds/2271
Sorry, Diana On 6 February 2017 at 10:07, Dylan McKay via cfe-commits <cfe-commits@lists.llvm.org> wrote: > Author: dylanmckay > Date: Mon Feb 6 03:07:56 2017 > New Revision: 294177 > > URL: http://llvm.org/viewvc/llvm-project?rev=294177&view=rev > Log: > [AVR] Allow specifying the CPU on the command line > > Summary: > This tells clang about all of the different AVR microcontrollers. > > It also adds code to define the correct preprocessor macros for each > device. > > Reviewers: jroelofs, asl > > Reviewed By: asl > > Subscribers: asl, cfe-commits > > Differential Revision: https://reviews.llvm.org/D28346 > > Added: > cfe/trunk/test/CodeGen/avr/ > cfe/trunk/test/CodeGen/avr/target-cpu-defines/ > cfe/trunk/test/CodeGen/avr/target-cpu-defines/atmega328p.c > cfe/trunk/test/CodeGen/avr/target-cpu-defines/attiny104.c > cfe/trunk/test/CodeGen/avr/target-cpu-defines/common.c > Modified: > cfe/trunk/lib/Basic/Targets.cpp > > Modified: cfe/trunk/lib/Basic/Targets.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/Targets.cpp?rev=294177&r1=294176&r2=294177&view=diff > ============================================================================== > --- cfe/trunk/lib/Basic/Targets.cpp (original) > +++ cfe/trunk/lib/Basic/Targets.cpp Mon Feb 6 03:07:56 2017 > @@ -8435,6 +8435,254 @@ public: > } > }; > > +/// Information about a specific microcontroller. > +struct MCUInfo { > + const char *Name; > + const char *DefineName; > +}; > + > +// This list should be kept up-to-date with AVRDevices.td in LLVM. > +static ArrayRef<MCUInfo> AVRMcus = { > + { "at90s1200", "__AVR_AT90S1200__" }, > + { "attiny11", "__AVR_ATtiny11" }, > + { "attiny12", "__AVR_ATtiny12" }, > + { "attiny15", "__AVR_ATtiny15" }, > + { "attiny28", "__AVR_ATtiny28" }, > + { "at90s2313", "__AVR_AT90S2313" }, > + { "at90s2323", "__AVR_AT90S2323" }, > + { "at90s2333", "__AVR_AT90S2333" }, > + { "at90s2343", "__AVR_AT90S2343" }, > + { "attiny22", "__AVR_ATtiny22" }, > + { "attiny26", "__AVR_ATtiny26" }, > + { "at86rf401", "__AVR_AT86RF401" }, > + { "at90s4414", "__AVR_AT90S4414" }, > + { "at90s4433", "__AVR_AT90S4433" }, > + { "at90s4434", "__AVR_AT90S4434" }, > + { "at90s8515", "__AVR_AT90S8515" }, > + { "at90c8534", "__AVR_AT90c8534" }, > + { "at90s8535", "__AVR_AT90S8535" }, > + { "ata5272", "__AVR_ATA5272" }, > + { "attiny13", "__AVR_ATtiny13" }, > + { "attiny13a", "__AVR_ATtiny13A" }, > + { "attiny2313", "__AVR_ATtiny2313" }, > + { "attiny2313a", "__AVR_ATtiny2313A" }, > + { "attiny24", "__AVR_ATtiny24" }, > + { "attiny24a", "__AVR_ATtiny24A" }, > + { "attiny4313", "__AVR_ATtiny4313" }, > + { "attiny44", "__AVR_ATtiny44" }, > + { "attiny44a", "__AVR_ATtiny44A" }, > + { "attiny84", "__AVR_ATtiny84" }, > + { "attiny84a", "__AVR_ATtiny84A" }, > + { "attiny25", "__AVR_ATtiny25" }, > + { "attiny45", "__AVR_ATtiny45" }, > + { "attiny85", "__AVR_ATtiny85" }, > + { "attiny261", "__AVR_ATtiny261" }, > + { "attiny261a", "__AVR_ATtiny261A" }, > + { "attiny461", "__AVR_ATtiny461" }, > + { "attiny461a", "__AVR_ATtiny461A" }, > + { "attiny861", "__AVR_ATtiny861" }, > + { "attiny861a", "__AVR_ATtiny861A" }, > + { "attiny87", "__AVR_ATtiny87" }, > + { "attiny43u", "__AVR_ATtiny43U" }, > + { "attiny48", "__AVR_ATtiny48" }, > + { "attiny88", "__AVR_ATtiny88" }, > + { "attiny828", "__AVR_ATtiny828" }, > + { "at43usb355", "__AVR_AT43USB355" }, > + { "at76c711", "__AVR_AT76C711" }, > + { "atmega103", "__AVR_ATmega103" }, > + { "at43usb320", "__AVR_AT43USB320" }, > + { "attiny167", "__AVR_ATtiny167" }, > + { "at90usb82", "__AVR_AT90USB82" }, > + { "at90usb162", "__AVR_AT90USB162" }, > + { "ata5505", "__AVR_ATA5505" }, > + { "atmega8u2", "__AVR_ATmega8U2" }, > + { "atmega16u2", "__AVR_ATmega16U2" }, > + { "atmega32u2", "__AVR_ATmega32U2" }, > + { "attiny1634", "__AVR_ATtiny1634" }, > + { "atmega8", "__AVR_ATmega8" }, > + { "ata6289", "__AVR_ATA6289" }, > + { "atmega8a", "__AVR_ATmega8A" }, > + { "ata6285", "__AVR_ATA6285" }, > + { "ata6286", "__AVR_ATA6286" }, > + { "atmega48", "__AVR_ATmega48" }, > + { "atmega48a", "__AVR_ATmega48A" }, > + { "atmega48pa", "__AVR_ATmega48PA" }, > + { "atmega48p", "__AVR_ATmega48P" }, > + { "atmega88", "__AVR_ATmega88" }, > + { "atmega88a", "__AVR_ATmega88A" }, > + { "atmega88p", "__AVR_ATmega88P" }, > + { "atmega88pa", "__AVR_ATmega88PA" }, > + { "atmega8515", "__AVR_ATmega8515" }, > + { "atmega8535", "__AVR_ATmega8535" }, > + { "atmega8hva", "__AVR_ATmega8HVA" }, > + { "at90pwm1", "__AVR_AT90PWM1" }, > + { "at90pwm2", "__AVR_AT90PWM2" }, > + { "at90pwm2b", "__AVR_AT90PWM2B" }, > + { "at90pwm3", "__AVR_AT90PWM3" }, > + { "at90pwm3b", "__AVR_AT90PWM3B" }, > + { "at90pwm81", "__AVR_AT90PWM81" }, > + { "ata5790", "__AVR_ATA5790" }, > + { "ata5795", "__AVR_ATA5795" }, > + { "atmega16", "__AVR_ATmega16" }, > + { "atmega16a", "__AVR_ATmega16A" }, > + { "atmega161", "__AVR_ATmega161" }, > + { "atmega162", "__AVR_ATmega162" }, > + { "atmega163", "__AVR_ATmega163" }, > + { "atmega164a", "__AVR_ATmega164A" }, > + { "atmega164p", "__AVR_ATmega164P" }, > + { "atmega164pa", "__AVR_ATmega164PA" }, > + { "atmega165", "__AVR_ATmega165" }, > + { "atmega165a", "__AVR_ATmega165A" }, > + { "atmega165p", "__AVR_ATmega165P" }, > + { "atmega165pa", "__AVR_ATmega165PA" }, > + { "atmega168", "__AVR_ATmega168" }, > + { "atmega168a", "__AVR_ATmega168A" }, > + { "atmega168p", "__AVR_ATmega168P" }, > + { "atmega168pa", "__AVR_ATmega168PA" }, > + { "atmega169", "__AVR_ATmega169" }, > + { "atmega169a", "__AVR_ATmega169A" }, > + { "atmega169p", "__AVR_ATmega169P" }, > + { "atmega169pa", "__AVR_ATmega169PA" }, > + { "atmega32", "__AVR_ATmega32" }, > + { "atmega32a", "__AVR_ATmega32A" }, > + { "atmega323", "__AVR_ATmega323" }, > + { "atmega324a", "__AVR_ATmega324A" }, > + { "atmega324p", "__AVR_ATmega324P" }, > + { "atmega324pa", "__AVR_ATmega324PA" }, > + { "atmega325", "__AVR_ATmega325" }, > + { "atmega325a", "__AVR_ATmega325A" }, > + { "atmega325p", "__AVR_ATmega325P" }, > + { "atmega325pa", "__AVR_ATmega325PA" }, > + { "atmega3250", "__AVR_ATmega3250" }, > + { "atmega3250a", "__AVR_ATmega3250A" }, > + { "atmega3250p", "__AVR_ATmega3250P" }, > + { "atmega3250pa", "__AVR_ATmega3250PA" }, > + { "atmega328", "__AVR_ATmega328" }, > + { "atmega328p", "__AVR_ATmega328P" }, > + { "atmega329", "__AVR_ATmega329" }, > + { "atmega329a", "__AVR_ATmega329A" }, > + { "atmega329p", "__AVR_ATmega329P" }, > + { "atmega329pa", "__AVR_ATmega329PA" }, > + { "atmega3290", "__AVR_ATmega3290" }, > + { "atmega3290a", "__AVR_ATmega3290A" }, > + { "atmega3290p", "__AVR_ATmega3290P" }, > + { "atmega3290pa", "__AVR_ATmega3290PA" }, > + { "atmega406", "__AVR_ATmega406" }, > + { "atmega64", "__AVR_ATmega64" }, > + { "atmega64a", "__AVR_ATmega64A" }, > + { "atmega640", "__AVR_ATmega640" }, > + { "atmega644", "__AVR_ATmega644" }, > + { "atmega644a", "__AVR_ATmega644A" }, > + { "atmega644p", "__AVR_ATmega644P" }, > + { "atmega644pa", "__AVR_ATmega644PA" }, > + { "atmega645", "__AVR_ATmega645" }, > + { "atmega645a", "__AVR_ATmega645A" }, > + { "atmega645p", "__AVR_ATmega645P" }, > + { "atmega649", "__AVR_ATmega649" }, > + { "atmega649a", "__AVR_ATmega649A" }, > + { "atmega649p", "__AVR_ATmega649P" }, > + { "atmega6450", "__AVR_ATmega6450" }, > + { "atmega6450a", "__AVR_ATmega6450A" }, > + { "atmega6450p", "__AVR_ATmega6450P" }, > + { "atmega6490", "__AVR_ATmega6490" }, > + { "atmega6490a", "__AVR_ATmega6490A" }, > + { "atmega6490p", "__AVR_ATmega6490P" }, > + { "atmega64rfr2", "__AVR_ATmega64RFR2" }, > + { "atmega644rfr2", "__AVR_ATmega644RFR2" }, > + { "atmega16hva", "__AVR_ATmega16HVA" }, > + { "atmega16hva2", "__AVR_ATmega16HVA2" }, > + { "atmega16hvb", "__AVR_ATmega16HVB" }, > + { "atmega16hvbrevb", "__AVR_ATmega16HVBREVB" }, > + { "atmega32hvb", "__AVR_ATmega32HVB" }, > + { "atmega32hvbrevb", "__AVR_ATmega32HVBREVB" }, > + { "atmega64hve", "__AVR_ATmega64HVE" }, > + { "at90can32", "__AVR_AT90CAN32" }, > + { "at90can64", "__AVR_AT90CAN64" }, > + { "at90pwm161", "__AVR_AT90PWM161" }, > + { "at90pwm216", "__AVR_AT90PWM216" }, > + { "at90pwm316", "__AVR_AT90PWM316" }, > + { "atmega32c1", "__AVR_ATmega32C1" }, > + { "atmega64c1", "__AVR_ATmega64C1" }, > + { "atmega16m1", "__AVR_ATmega16M1" }, > + { "atmega32m1", "__AVR_ATmega32M1" }, > + { "atmega64m1", "__AVR_ATmega64M1" }, > + { "atmega16u4", "__AVR_ATmega16U4" }, > + { "atmega32u4", "__AVR_ATmega32U4" }, > + { "atmega32u6", "__AVR_ATmega32U6" }, > + { "at90usb646", "__AVR_AT90USB646" }, > + { "at90usb647", "__AVR_AT90USB647" }, > + { "at90scr100", "__AVR_AT90SCR100" }, > + { "at94k", "__AVR_AT94K" }, > + { "m3000", "__AVR_AT000" }, > + { "atmega128", "__AVR_ATmega128" }, > + { "atmega128a", "__AVR_ATmega128A" }, > + { "atmega1280", "__AVR_ATmega1280" }, > + { "atmega1281", "__AVR_ATmega1281" }, > + { "atmega1284", "__AVR_ATmega1284" }, > + { "atmega1284p", "__AVR_ATmega1284P" }, > + { "atmega128rfa1", "__AVR_ATmega128RFA1" }, > + { "atmega128rfr2", "__AVR_ATmega128RFR2" }, > + { "atmega1284rfr2", "__AVR_ATmega1284RFR2" }, > + { "at90can128", "__AVR_AT90CAN128" }, > + { "at90usb1286", "__AVR_AT90USB1286" }, > + { "at90usb1287", "__AVR_AT90USB1287" }, > + { "atmega2560", "__AVR_ATmega2560" }, > + { "atmega2561", "__AVR_ATmega2561" }, > + { "atmega256rfr2", "__AVR_ATmega256RFR2" }, > + { "atmega2564rfr2", "__AVR_ATmega2564RFR2" }, > + { "atxmega16a4", "__AVR_ATxmega16A4" }, > + { "atxmega16a4u", "__AVR_ATxmega16a4U" }, > + { "atxmega16c4", "__AVR_ATxmega16C4" }, > + { "atxmega16d4", "__AVR_ATxmega16D4" }, > + { "atxmega32a4", "__AVR_ATxmega32A4" }, > + { "atxmega32a4u", "__AVR_ATxmega32A4U" }, > + { "atxmega32c4", "__AVR_ATxmega32C4" }, > + { "atxmega32d4", "__AVR_ATxmega32D4" }, > + { "atxmega32e5", "__AVR_ATxmega32E5" }, > + { "atxmega16e5", "__AVR_ATxmega16E5" }, > + { "atxmega8e5", "__AVR_ATxmega8E5" }, > + { "atxmega32x1", "__AVR_ATxmega32X1" }, > + { "atxmega64a3", "__AVR_ATxmega64A3" }, > + { "atxmega64a3u", "__AVR_ATxmega64A3U" }, > + { "atxmega64a4u", "__AVR_ATxmega64A4U" }, > + { "atxmega64b1", "__AVR_ATxmega64B1" }, > + { "atxmega64b3", "__AVR_ATxmega64B3" }, > + { "atxmega64c3", "__AVR_ATxmega64C3" }, > + { "atxmega64d3", "__AVR_ATxmega64D3" }, > + { "atxmega64d4", "__AVR_ATxmega64D4" }, > + { "atxmega64a1", "__AVR_ATxmega64A1" }, > + { "atxmega64a1u", "__AVR_ATxmega64A1U" }, > + { "atxmega128a3", "__AVR_ATxmega128A3" }, > + { "atxmega128a3u", "__AVR_ATxmega128A3U" }, > + { "atxmega128b1", "__AVR_ATxmega128B1" }, > + { "atxmega128b3", "__AVR_ATxmega128B3" }, > + { "atxmega128c3", "__AVR_ATxmega128C3" }, > + { "atxmega128d3", "__AVR_ATxmega128D3" }, > + { "atxmega128d4", "__AVR_ATxmega128D4" }, > + { "atxmega192a3", "__AVR_ATxmega192A3" }, > + { "atxmega192a3u", "__AVR_ATxmega192A3U" }, > + { "atxmega192c3", "__AVR_ATxmega192C3" }, > + { "atxmega192d3", "__AVR_ATxmega192D3" }, > + { "atxmega256a3", "__AVR_ATxmega256A3" }, > + { "atxmega256a3u", "__AVR_ATxmega256A3U" }, > + { "atxmega256a3b", "__AVR_ATxmega256A3B" }, > + { "atxmega256a3bu", "__AVR_ATxmega256A3BU" }, > + { "atxmega256c3", "__AVR_ATxmega256C3" }, > + { "atxmega256d3", "__AVR_ATxmega256D3" }, > + { "atxmega384c3", "__AVR_ATxmega384C3" }, > + { "atxmega384d3", "__AVR_ATxmega384D3" }, > + { "atxmega128a1", "__AVR_ATxmega128A1" }, > + { "atxmega128a1u", "__AVR_ATxmega128A1U" }, > + { "atxmega128a4u", "__AVR_ATxmega128a4U" }, > + { "attiny4", "__AVR_ATtiny4" }, > + { "attiny5", "__AVR_ATtiny5" }, > + { "attiny9", "__AVR_ATtiny9" }, > + { "attiny10", "__AVR_ATtiny10" }, > + { "attiny20", "__AVR_ATtiny20" }, > + { "attiny40", "__AVR_ATtiny40" }, > + { "attiny102", "__AVR_ATtiny102" }, > + { "attiny104", "__AVR_ATtiny104" }, > +}; > > // AVR Target > class AVRTargetInfo : public TargetInfo { > @@ -8476,7 +8724,17 @@ public: > > void getTargetDefines(const LangOptions &Opts, > MacroBuilder &Builder) const override { > + Builder.defineMacro("AVR"); > + Builder.defineMacro("__AVR"); > Builder.defineMacro("__AVR__"); > + > + if (!this->CPU.empty()) { > + auto It = std::find_if(AVRMcus.begin(), AVRMcus.end(), > + [&](const MCUInfo &Info) { return Info.Name == this->CPU; }); > + > + if (It != AVRMcus.end()) > + Builder.defineMacro(It->DefineName); > + } > } > > ArrayRef<Builtin::Info> getTargetBuiltins() const override { > @@ -8585,6 +8843,41 @@ public: > ? (IsSigned ? SignedInt : UnsignedInt) > : TargetInfo::getLeastIntTypeByWidth(BitWidth, IsSigned); > } > + > + bool setCPU(const std::string &Name) override { > + bool IsFamily = llvm::StringSwitch<bool>(Name) > + .Case("avr1", true) > + .Case("avr2", true) > + .Case("avr25", true) > + .Case("avr3", true) > + .Case("avr31", true) > + .Case("avr35", true) > + .Case("avr4", true) > + .Case("avr5", true) > + .Case("avr51", true) > + .Case("avr6", true) > + .Case("avrxmega1", true) > + .Case("avrxmega2", true) > + .Case("avrxmega3", true) > + .Case("avrxmega4", true) > + .Case("avrxmega5", true) > + .Case("avrxmega6", true) > + .Case("avrxmega7", true) > + .Case("avrtiny", true) > + .Default(false); > + > + if (IsFamily) this->CPU = Name; > + > + bool IsMCU = std::find_if(AVRMcus.begin(), AVRMcus.end(), > + [&](const MCUInfo &Info) { return Info.Name == Name; }) != > AVRMcus.end(); > + > + if (IsMCU) this->CPU = Name; > + > + return IsFamily || IsMCU; > + } > + > +protected: > + std::string CPU; > }; > > } // end anonymous namespace > > Added: cfe/trunk/test/CodeGen/avr/target-cpu-defines/atmega328p.c > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGen/avr/target-cpu-defines/atmega328p.c?rev=294177&view=auto > ============================================================================== > --- cfe/trunk/test/CodeGen/avr/target-cpu-defines/atmega328p.c (added) > +++ cfe/trunk/test/CodeGen/avr/target-cpu-defines/atmega328p.c Mon Feb 6 > 03:07:56 2017 > @@ -0,0 +1,6 @@ > +// RUN: %clang_cc1 -E -dM -triple avr-unknown-unknown -target-cpu atmega328p > /dev/null | FileCheck -match-full-lines %s > + > +// CHECK: #define AVR 1 > +// CHECK: #define __AVR 1 > +// CHECK: #define __AVR_ATmega328P 1 > +// CHECK: #define __AVR__ 1 > > Added: cfe/trunk/test/CodeGen/avr/target-cpu-defines/attiny104.c > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGen/avr/target-cpu-defines/attiny104.c?rev=294177&view=auto > ============================================================================== > --- cfe/trunk/test/CodeGen/avr/target-cpu-defines/attiny104.c (added) > +++ cfe/trunk/test/CodeGen/avr/target-cpu-defines/attiny104.c Mon Feb 6 > 03:07:56 2017 > @@ -0,0 +1,6 @@ > +// RUN: %clang_cc1 -E -dM -triple avr-unknown-unknown -target-cpu attiny104 > /dev/null | FileCheck -match-full-lines %s > + > +// CHECK: #define AVR 1 > +// CHECK: #define __AVR 1 > +// CHECK: #define __AVR_ATtiny104 1 > +// CHECK: #define __AVR__ 1 > > Added: cfe/trunk/test/CodeGen/avr/target-cpu-defines/common.c > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGen/avr/target-cpu-defines/common.c?rev=294177&view=auto > ============================================================================== > --- cfe/trunk/test/CodeGen/avr/target-cpu-defines/common.c (added) > +++ cfe/trunk/test/CodeGen/avr/target-cpu-defines/common.c Mon Feb 6 > 03:07:56 2017 > @@ -0,0 +1,5 @@ > +// RUN: %clang_cc1 -E -dM -triple avr-unknown-unknown /dev/null | FileCheck > -match-full-lines %s > + > +// CHECK: #define AVR 1 > +// CHECK: #define __AVR 1 > +// CHECK: #define __AVR__ 1 > > > _______________________________________________ > cfe-commits mailing list > cfe-commits@lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits