[PATCH] D123200: [compiler-rt][builtins] Add several helper functions for AVR

[Fangrui Song via Phabricator via cfe-commits]

MaskRay accepted this revision.
MaskRay added inline comments.
This revision is now accepted and ready to land.


================
Comment at: compiler-rt/cmake/Modules/CompilerRTUtils.cmake:171
   check_symbol_exists(__ve__ "" __VE)
+  check_symbol_exists(__AVR__ "" __AVR)
   if(__ARM)
----------------
Keep the list in alphabetical order. Move avr to the beginning. Ignore some entries which are unordered.


================
Comment at: compiler-rt/cmake/Modules/CompilerRTUtils.cmake:216
     add_default_target_arch(ve)
+  elseif(__AVR)
+    add_default_target_arch(avr)
----------------
Keep the list in alphabetical order. Move avr to the beginning. Ignore some entries which are unordered.


================
Comment at: compiler-rt/cmake/builtin-config-ix.cmake:55
 set(VE ve)
+set(AVR avr)
 
----------------
Keep the list in alphabetical order. Move avr to the beginning. Ignore some entries which are unordered.


================
Comment at: compiler-rt/lib/builtins/CMakeLists.txt:671
 
+set(avr_SOURCES
+  avr/mulqi3.S
----------------
Keep the `*_SOURCES` in alphabetical order. Move avr to the beginning. Ignore some entries which are unordered.


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

https://reviews.llvm.org/D123200