[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