commit d52adb9ccd98fd37a081bfc4fe1d3b5634959203
author Brian Osman <brianosman@google.com> Wed Mar 13 13:44:24 2024 -0400
committer SkCQ <skcq-be@skia-corp.google.com.iam.gserviceaccount.com> Wed Mar 13 18:01:52 2024 +0000
tree 1262470609193d7431ffc112fb14a59214d3ad5c
parent b440ca2fa89c9d03258c2997680fbb5294ab7c38

Disable musttail on LoongArch

Developers contributing LoongArch support have indicated that they had
to disable musttail

Change-Id: I5aaca3c45517d581078b197c85a34da2b170bca3
Reviewed-on: https://skia-review.googlesource.com/c/skcms/+/825726
Auto-Submit: Brian Osman <brianosman@google.com>
Commit-Queue: John Stiles <johnstiles@google.com>
Reviewed-by: John Stiles <johnstiles@google.com>

src/skcms_internals.h

diff --git a/src/skcms_internals.h b/src/skcms_internals.h
index 3871ea3..f3f0a2d 100644
--- a/src/skcms_internals.h
+++ b/src/skcms_internals.h
@@ -39,6 +39,7 @@
         // - Clang 18 runs into an ICE on armv7/androideabi with [[clang::musttail]].
         //   (http://crbug.com/1504548)
         // - Android RISC-V also runs into an ICE (b/314692534)
+        // - LoongArch developers indicate they had to turn it off
         // - Windows builds generate incorrect code with [[clang::musttail]] and crash mysteriously.
         //   (http://crbug.com/1505442)
         #if __has_cpp_attribute(clang::musttail) && !__has_feature(memory_sanitizer) \
@@ -46,6 +47,7 @@
                                                  && !defined(__EMSCRIPTEN__) \
                                                  && !defined(__arm__) \
                                                  && !defined(__riscv) \
+                                                 && !defined(__loongarch__) \
                                                  && !defined(_WIN32) && !defined(__SYMBIAN32__)
             #define SKCMS_HAS_MUSTTAIL 1
         #endif