[Interp] Mark the test unsupported with Asan

[vitalybuka]

It's very flaky recently.

10 bulds are OK, then 3 failed:
https://lab.llvm.org/buildbot/#/builders/52/builds/1524
https://lab.llvm.org/buildbot/#/builders/52/builds/1525
https://lab.llvm.org/buildbot/#/builders/52/builds/1526
then 3 OK
https://lab.llvm.org/buildbot/#/builders/52/builds/1531
https://lab.llvm.org/buildbot/#/builders/52/builds/1532
then 2 green again

We need to stop to spam blame list by disabling the test,
and investigate later if Asan is valuable for the test.

Issue #102858

[llvmbot]

@llvm/pr-subscribers-clang

Author: Vitaly Buka (vitalybuka)

Changes

It's very flaky recently.

Issue #102858

---

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

1 Files Affected:

• (modified) clang/test/Interpreter/const.cpp (+3)

diff --git a/clang/test/Interpreter/const.cpp b/clang/test/Interpreter/const.cpp
index 57fd880400e6a1..6cafc777c626e7 100644
--- a/clang/test/Interpreter/const.cpp
+++ b/clang/test/Interpreter/const.cpp
@@ -2,6 +2,9 @@
 // see https://github.com/llvm/llvm-project/issues/68092
 // XFAIL: host={{.*}}-windows-msvc
 
+// The test is flaky with asan.
+// UNSUPPORTED: asan
+
 // RUN: cat %s | clang-repl | FileCheck %s
 // RUN: cat %s | clang-repl -Xcc -O2 | FileCheck %s
 

[hahnjo]

Quick question for my understanding: With spr we don't get meaningful commit messages anymore? That's quite unfortunate...

[vitalybuka]

Quick question for my understanding: With spr we don't get meaningful commit messages anymore? That's quite unfortunate...

Can you please clarify?

If I understand you questing correctly, then nothing is special about spr, it puts what ever was in the commit.

If this cryptic request for more details :), then my preference to keep details in the issue, I don't see value in noise from pasted errors into commit log.

Added some details, but I don't really have much to add, and build links will expire in a couple of weeks.

[yronglin]

LGTM

[hahnjo]

Quick question for my understanding: With spr we don't get meaningful commit messages anymore? That's quite unfortunate...

Can you please clarify?

The commit messages in this PR are initial version and Update const.cpp, which is totally useless if I came across this in git log. I meanwhile remembered that the policy in LLVM is to squash-and-merge, so I guess you can supply a more useful message at that point, but I would still argue that the history deserves better than what is currently in this PR.

[vitalybuka]

Quick question for my understanding: With spr we don't get meaningful commit messages anymore? That's quite unfortunate...

Can you please clarify?

The commit messages in this PR are initial version and Update const.cpp, which is totally useless if I came across this in git log. I meanwhile remembered that the policy in LLVM is to squash-and-merge, so I guess you can supply a more useful message at that point, but I would still argue that the history deserves better than what is currently in this PR.

LLVM repo does not allow "merge" or "rebase", everything is squash, with ONLY commit message created from the description of the PR.
With or without spr internal PR commits description are irrelevant.

[vitalybuka]

@hahnjo I believe I explained "Update const.cpp" issue.
So I am going to merge as the that continue to sporadically brake.