[Mlir-commits] [mlir] [memref] Support dense resources for memref-to-llvm (PR #79380)

[Kunwar Grover]

Groverkss wrote:

> > For a big-endian system, this may be differently interpreted and there would be a different dense<> attribute. So the test output actually depends on the endianness of the system.
> 
> To be clear: that's a bug though right?

No, it's not a bug. dense_resource does not specify what endianness to use. So the best we can do is assume the endianness of the system (which is what we do). So this test simply has different outputs on a big endian machine.

The expectation for dense_resource, AFAIU, is that whoever created it should read it.

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