[llvm] [NFC] Remove `reflect-error.ll` as a failing testcase (PR #124037)

Chris B via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 22 17:04:21 PST 2025


https://github.com/llvm-beanz requested changes to this pull request.

I don't think we should be deleting a test because it surfaced a bug, and I definitely don't think we should accept that the SPIRV backend not having a practice of testing its error conditions means we shouldn't test them.

We should test error conditions, and we should understand why hwasan is failing on this test because it is likely a bug that we need to address.

https://github.com/llvm/llvm-project/pull/124037


More information about the llvm-commits mailing list