Pavel Chupin
Pavel Chupin
ping @intel/dpcpp-specification-reviewers
@KornevNikita please check Windows post-commit fail: https://github.com/intel/llvm/actions/runs/3107743966/jobs/5036187282
@zahiraam, please follow up on post-commit issue on Windows: https://github.com/intel/llvm/actions/runs/3153782586 ``` Failed Tests (1): Clang :: CodeGenSYCL/debug-info-file-checksum.cpp ```
@zahiraam, I'm getting the same crash locally. I'm using MSVC2022 if it helps. Would you like me to revert the patch?
@zahiraam, here is the backtrace from debug version: ``` Expected must be checked before access or destruction. Unchecked Expected contained error: invalid argumentPLEASE submit a bug report to https://github.com/llvm/llvm-project/issues/ and...
@zahiraam, sounds good to me. Should I revert this change as a whole to give you more time for analysis? Or you have a workaround? Or should we disable the...
@elizabethandrews, can you follow up on post commit issues on windows? https://github.com/intel/llvm/actions/runs/3137630364/jobs/5096023697
@jinge90, please address @romanovvlad comments. And please get @intel/llvm-reviewers-runtime approval.
> Could you please add more details to the PR summary? @jinge90, please do this.
It looks good to me but would be good to see more reviews/comments from @intel/llvm-reviewers-runtime who write the tests and expected to be compliant with this doc.