[PATCH] D28346: [AVR] Allow specifying the CPU on the command line
Anton Korobeynikov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Feb 6 00:55:48 PST 2017
asl accepted this revision.
asl added a comment.
This revision is now accepted and ready to land.
LGTM with minor style nit
================
Comment at: lib/Basic/Targets.cpp:8733
+ auto It = std::find_if(AVRMcus.begin(),
+ AVRMcus.end(),
+ [&](const MCUInfo &Info) { return Info.Name == this->CPU; });
----------------
Minor style nit: I'd prefer end() to be on prev. line (like below)
https://reviews.llvm.org/D28346
More information about the cfe-commits
mailing list