[PATCH] D137572: [AVR][Clang] Implement __AVR_HAVE_*__ macros

Ben Shi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 11 02:03:40 PST 2022


benshi001 added inline comments.


================
Comment at: clang/lib/Basic/Targets/AVR.cpp:355
+    .Cases("31", "51", "6", true)
+    .Cases("104", "105", "106", "107", true)
+    .Default(false);
----------------
ATxmega16a4 with family code 102 also supports ELPM. Could you please make a careful check on ELPM and all other features? 

Generally speaking I am very glad to have this patch committed, since it fixes

https://github.com/llvm/llvm-project/issues/56157



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137572/new/

https://reviews.llvm.org/D137572



More information about the cfe-commits mailing list