sdkrystian wrote:

> Yup, that fixes the issue. I've also confirmed that reverting the fix brings 
> back the crash!

Great!

> It is weird that you were unable to repro this 🤔 (or rather that it was also 
> crashing when you reverted the original patch...) Are you 100% sure you 
> recompiled LLDB after reverting?

So... I usually compile clang with msvc on windows (I would use clang-cl if 
VS2022 didn't hang when debugging). I've spent many hours trying to get LLDB to 
compile using my current environment without success, so I instead compile in 
WSL. Since 
`lldb/test/API/commands/expression/import-std-module/array/TestArrayFromStdModule.py`
 is unsupported on Linux, I just commented out `@skipIfLinux` so the test will 
at least run. The test fails for me locally, but when the patch is applied it 
doesn't crash.

https://github.com/llvm/llvm-project/pull/114258
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to