[MSVC] work-around for compile time issue 102513

[bd1976bris]

Manual cherry-pick of #110986 to the LLVM 19 release branch.

[llvmbot]

@llvm/pr-subscribers-clang

Author: bd1976bris (bd1976bris)

Changes

Manual cherry-pick of #110986 to the LLVM 19 release branch.

---

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

1 Files Affected:

• (modified) clang/lib/AST/Interp/Interp.cpp (+8)

diff --git a/clang/lib/AST/Interp/Interp.cpp b/clang/lib/AST/Interp/Interp.cpp
index 6fcd90e5f58498..0f9eedc3f38ea1 100644
--- a/clang/lib/AST/Interp/Interp.cpp
+++ b/clang/lib/AST/Interp/Interp.cpp
@@ -925,6 +925,10 @@ void diagnoseEnumValue(InterpState &S, CodePtr OpPC, const EnumDecl *ED,
   }
 }
 
+// https://github.com/llvm/llvm-project/issues/102513
+#if defined(_WIN32) && !defined(__clang__) && !defined(NDEBUG)
+#pragma optimize("", off)
+#endif
 bool Interpret(InterpState &S, APValue &Result) {
   // The current stack frame when we started Interpret().
   // This is being used by the ops to determine wheter
@@ -949,6 +953,10 @@ bool Interpret(InterpState &S, APValue &Result) {
     }
   }
 }
+// https://github.com/llvm/llvm-project/issues/102513
+#if defined(_WIN32) && !defined(__clang__) && !defined(NDEBUG)
+#pragma optimize("", on)
+#endif
 
 } // namespace interp
 } // namespace clang

[bd1976bris]

@tru I'm unable to merge this even with approval. Perhaps this has been made against the wrong branch or 19.x now closed?

[tru]

@tru I'm unable to merge this even with approval. Perhaps this has been made against the wrong branch or 19.x now closed?

No it's correct. Only the release manager can merge to the release branch, I will do that before the next release.

[bd1976bris]

@tru I'm unable to merge this even with approval. Perhaps this has been made against the wrong branch or 19.x now closed?

No it's correct. Only the release manager can merge to the release branch, I will do that before the next release.

Thanks very much!

[bd1976bris]

@tru should this have been merged? Do I need to do something to facilitate? Sorry for not following up earlier I have been sick recently. Thanks.

[nikic]

@tru should this have been merged? Do I need to do something to facilitate? Sorry for not following up earlier I have been sick recently. Thanks.

Backport PRs need to be part of the release milestone, otherwise they're likely to get forgotten about :) I added it just now.

[github-actions]

@bd1976bris (or anyone else). If you would like to add a note about this fix in the release notes (completely optional). Please reply to this comment with a one or two sentence description of the fix. When you are done, please add the release:note label to this PR.