[PATCH] D111703: [ARM] __cxa_end_cleanup should be called instead of _UnwindResume.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 25 09:33:24 PDT 2021


MaskRay added inline comments.


================
Comment at: llvm/include/llvm/ADT/Triple.h:735
+            getEnvironment() == Triple::MuslEABIHF || isAndroid()) &&
+           !isOSDarwin() && !isOSWindows();
+  }
----------------
Can Darwin or Windows use EHABI related environment? If not, remove `!isOSDarwin() && !isOSWindows()`.
If possible, check isOSBinFormatELF() instead


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

https://reviews.llvm.org/D111703



More information about the llvm-commits mailing list