xgupta wrote:

> The test in issue #146335 doesn't seem to be crashing for me! You are using 
> it as a LIT test for the patch. Shouldn't there be a use of a `parallel-for` 
> as the issue's title suggest?


I tried today and it seems test case is not crashing anymore - 
```
~/llvm-project/build (pr146335*) » clang++ -std=c++20  -fopenmp -c test.cpp -o 
test1                                                                           
                                       
test.cpp:52:20: warning: field 'q' is uninitialized when used here 
[-Wuninitialized]
   52 |   H(n ag, m r) : D(q, 0, ag, r, v), t{o(af, 0, 0)} {}
      |                    ^
test.cpp:52:33: warning: field 'v' is uninitialized when used here 
[-Wuninitialized]
   52 |   H(n ag, m r) : D(q, 0, ag, r, v), t{o(af, 0, 0)} {}
      |                                 ^
test.cpp:52:41: warning: field 'af' is uninitialized when used here 
[-Wuninitialized]
   52 |   H(n ag, m r) : D(q, 0, ag, r, v), t{o(af, 0, 0)} {}
      |                                         ^
3 warnings generated.
```

```
~/llvm-project/build (pr146335*) » ../install/bin/clang++ --version             
                                                                                
                                        
clang version 23.0.0git (https://github.com/xgupta/llvm-project.git 
f2060a946f1cebb4434d624a52a13dadfec7a08f)
Target: arm64-apple-darwin25.3.0
Thread model: posix
InstalledDir: /Users/xgupta/llvm-project/install/bin
Build config: +assertions
```

But I do seeing it is crashing on latest trunk on godbolt 
-https://godbolt.org/z/c38bW7Wv7.

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

Reply via email to