llvmbot wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Timm Baeder (tbaederr)

<details>
<summary>Changes</summary>

Apparently it causes problems there, too.
See https://lab.llvm.org/buildbot/#/builders/24/builds/18781

---
Full diff: https://github.com/llvm/llvm-project/pull/188042.diff


1 Files Affected:

- (modified) clang/lib/AST/ByteCode/Interp.cpp (+3-1) 


``````````diff
diff --git a/clang/lib/AST/ByteCode/Interp.cpp 
b/clang/lib/AST/ByteCode/Interp.cpp
index fc783fe4ec871..e2c9b01f4c7ea 100644
--- a/clang/lib/AST/ByteCode/Interp.cpp
+++ b/clang/lib/AST/ByteCode/Interp.cpp
@@ -40,7 +40,9 @@ using namespace clang::interp;
 // We disable it on MSVC generally since it doesn't seem to be able
 // to handle the way we use tailcalls.
 // PPC can't tail-call external calls, which is a problem for InterpNext.
-#if defined(_MSC_VER) || defined(_ARCH_PPC) || !defined(MUSTTAIL)
+// aarch64 seems to have problems too, at least with sanitizers enabled.
+#if defined(_MSC_VER) || defined(_ARCH_PPC) || defined(__aarch64__) ||         
\
+    !defined(MUSTTAIL)
 #define MUSTTAIL
 #define USE_TAILCALLS 0
 #else

``````````

</details>


https://github.com/llvm/llvm-project/pull/188042
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to