llvmorg-github-actions[bot] wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Jonas Rembser (guitargeek)

<details>
<summary>Changes</summary>

The test added in #&lt;PR of the void-expression fix&gt; checked the output of 
a JIT'd printf() with CHECK-NEXT between lines that clang-repl prints itself. 
The C runtime buffers the JIT'd code's stdout separately, so the order is not 
guaranteed: on llvm-clang-x86_64-sie-win, "foo()" came after all of 
clang-repl's output.

Remove that case. The case before it already checks that a void call without a 
semicolon is executed (through the value of a variable it sets), and so does 
the unit test ValueOfVoidCallExecutesTheCall.

Assisted-by: Claude Opus 5.5

Follows up on #<!-- -->228334, which introduced failures on Windows:

```
The Buildbot has detected a new failure on builder llvm-clang-x86_64-sie-win 
while building clang.

Full details are available at:
    https://lab.llvm.org/buildbot/#/builders/46/builds/42212

Worker for this Build: sie-win-worker
Blamelist:
    Frederik Harwath [&lt;frederik.harwath@<!-- 
-->amd.com&gt;](mailto:frederik.harwath@<!-- -->amd.com),
    Jonas Rembser [&lt;jonas.rembser@<!-- 
-->cern.ch&gt;](mailto:jonas.rembser@<!-- -->cern.ch),
    Lang Hames [&lt;lhames@<!-- -->gmail.com&gt;](mailto:lhames@<!-- 
-->gmail.com)

BUILD FAILED: failed test (failure)

Step 7 (test-build-unified-tree-check-all) failure: test (failure)
******************** TEST 'Clang :: Interpreter/pretty-print.cpp' FAILED 
********************
Exit Code: 1

Command Output (stdout):
--
# RUN: at line 1
z:\b\llvm-clang-x86_64-sie-win\build\bin\clang-repl.exe "int i = 10;" 'extern 
"C" int printf(const char*,...);'             'auto r1 = printf("i = %d\n", 
i);' | z:\b\llvm-clang-x86_64-sie-win\build\bin\filecheck.exe 
--check-prefix=CHECK-DRIVER 
Z:\b\llvm-clang-x86_64-sie-win\llvm-project\clang\test\Interpreter\pretty-print.cpp
# executed command: 'z:\b\llvm-clang-x86_64-sie-win\build\bin\clang-repl.exe' 
'int i = 10;' 'extern "C" int printf(const char*,...);' 'auto r1 = printf("i = 
%d\n", i);'
# note: command had no output on stdout or stderr
# executed command: 'z:\b\llvm-clang-x86_64-sie-win\build\bin\filecheck.exe' 
--check-prefix=CHECK-DRIVER 
'Z:\b\llvm-clang-x86_64-sie-win\llvm-project\clang\test\Interpreter\pretty-print.cpp'
# note: command had no output on stdout or stderr
# RUN: at line 6
cat 
Z:\b\llvm-clang-x86_64-sie-win\llvm-project\clang\test\Interpreter\pretty-print.cpp
 | z:\b\llvm-clang-x86_64-sie-win\build\bin\clang-repl.exe -Xcc -std=c++11 -Xcc 
-fno-delayed-template-parsing | 
z:\b\llvm-clang-x86_64-sie-win\build\bin\filecheck.exe 
Z:\b\llvm-clang-x86_64-sie-win\llvm-project\clang\test\Interpreter\pretty-print.cpp
# executed command: cat 
'Z:\b\llvm-clang-x86_64-sie-win\llvm-project\clang\test\Interpreter\pretty-print.cpp'
# note: command had no output on stdout or stderr
# executed command: 'z:\b\llvm-clang-x86_64-sie-win\build\bin\clang-repl.exe' 
-Xcc -std=c++11 -Xcc -fno-delayed-template-parsing
# note: command had no output on stdout or stderr
# executed command: 'z:\b\llvm-clang-x86_64-sie-win\build\bin\filecheck.exe' 
'Z:\b\llvm-clang-x86_64-sie-win\llvm-project\clang\test\Interpreter\pretty-print.cpp'
# .---command stderr------------
# | 
Z:\b\llvm-clang-x86_64-sie-win\llvm-project\clang\test\Interpreter\pretty-print.cpp:101:16:
 error: CHECK-NEXT: is not on the line after the previous 
match
# | // CHECK-NEXT: foo()
# |                ^
# | &lt;stdin&gt;:25:1: note: 'next' 
match was here
# | foo()
# | ^
# | &lt;stdin&gt;:23:10: note: 
previous match ended here
# | (int) 100
# |          ^
# | &lt;stdin&gt;:24:1: note: 
non-matching line after previous match is here
# | clang-repl&gt; clang-repl... clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-re
 pl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; clang-repl&gt; 
clang-repl&gt; clang-repl&gt; 
# | ^
# | 
# | Input file: &lt;stdin&gt;
# | Check file: 
Z:\b\llvm-clang-x86_64-sie-win\llvm-project\clang\test\Interpreter\pretty-print.cpp
# | 
# | -dump-input=help explains the following input dump.
# | 
# | Input was:
# | &lt;&lt;&lt;&lt;&lt;&lt;
# |             1: (const char[3]) 
"ab" 
# | check:10       ^~~~~~~~~~~~~~~~~~~~
# |             2: (int) 
123456 
# | next:13        ^~~~~~~~~~~~
# |             3: (char[2]) { 
'1', 'a' } 
# | next:16        ^~~~~~~~~~~~~~~~~~~~~~
# |             4: (char[3]) 
"1a" 
# | next:19        ^~~~~~~~~~~~~~
# |             5: 
(char[2][3][1]) { { { 'a' }, { 'b' }, { 'c' } }, { { 'd' 
}, { 'e' }, { 'f' } } } 
# | check:22       
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
# |             6: (S3) @<!-- 
-->0x1cf15f38540 
...

Sincerely,
LLVM Buildbot
```

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


1 Files Affected:

- (modified) clang/test/Interpreter/pretty-print.cpp (-3) 


``````````diff
diff --git a/clang/test/Interpreter/pretty-print.cpp 
b/clang/test/Interpreter/pretty-print.cpp
index ed430d2db6d22..777d809873203 100644
--- a/clang/test/Interpreter/pretty-print.cpp
+++ b/clang/test/Interpreter/pretty-print.cpp
@@ -96,9 +96,6 @@ void set_void_x() { void_x = 100; }
 set_void_x()
 void_x
 // CHECK-NEXT: (int) 100
-void print_foo() { printf("foo()\n"); }
-print_foo()
-// CHECK-NEXT: foo()
 
 // int i = 12;
 // int &iref = i;

``````````

</details>


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

Reply via email to