[llvm] [AMDGPU][GlobalISel] Fix load/store of pointer vectors, buffer.*.pN (PR #110714)

NAKAMURA Takumi via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 5 23:39:08 PDT 2024


chapuni wrote:

Reverted again.

I've investigated further. Tests will fail with `-DNDEBUG`.

`reportGISelFailure()` is called only in +Asserts.
https://github.com/llvm/llvm-project/commit/2b94972eb981090e0289ba56edd22b311e9fd6f7#diff-764f8c5ac9f21f864c5f677d103b945de05af957c7f993eebeb0bce8fbd25125R88
It marks a property.
https://github.com/llvm/llvm-project/commit/7c6ca18fffdcf935b3b38ffc8697180d60511ffe#diff-7b1e350b8a96379b42928d6d373d502796b98500bbfe87d3b7abbcc08e1b8bc5R228

A quick fix will be mark tests with `REQUIRES: asserts`.
I didn't notice why the result was affected by assertions.

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


More information about the llvm-commits mailing list