[clang] [clang][bytecode] Disable tail calls on MSVC (PR #188033)

[Timm Baeder via cfe-commits]

Timm =?utf-8?q?Bäder?= <tbaeder at redhat.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/188033 at github.com>


================
@@ -37,7 +37,9 @@ using namespace clang::interp;
 #endif
 
 // On MSVC, musttail does not guarantee tail calls in debug mode.
-#if (defined(_MSC_VER) && defined(_DEBUG)) || !defined(MUSTTAIL)
+// We disable it on MSVC generally since it doesn't seem to be able
+// to handle the way we use tailcalls.
+#if defined(_MSC_VER) || !defined(MUSTTAIL)
----------------
tbaederr wrote:

Doesn't hurt. Someone who can test this stuff can enable it for clang-cl later if they want, but I'm flying blind here.

https://github.com/llvm/llvm-project/pull/188033