[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