[PATCH] D80272: [mlir] Support big-endian systems in DenseElementsAttr (multiple word)

[Haruki Imai via Phabricator via llvm-commits]

imaihal added a comment.

In D80272#2090239 <https://reviews.llvm.org/D80272#2090239>, @rriddle wrote:

> Can you try Eli's suggestion in this patch?

@rriddle Sorry for late response. He suggested we should not check endianenss, This means we should use the same code in BE and LE machines. 
However, I think redundant copy are required in LE to use the same code.  So, current patch checks the endianness to avoid it.
Do you have any thought?

     


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80272/new/

https://reviews.llvm.org/D80272