[PATCH] D94964: [LangRef] Describe memory layout for vectors types
Danila Malyutin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 27 12:06:14 PDT 2023
danilaml added inline comments.
================
Comment at: llvm/docs/LangRef.rst:3253
+ ; [%ptr + 0]: 01010011 (0x53)
+ ; [%ptr + 1]: 00100001 (0x21)
+
----------------
bjope wrote:
> danilaml wrote:
> > Why is the least significant byte is placed at the largest memory address here, if it's little endian, just like for big endian?
> > Shouldn't it be reversed?
> @danilaml : Right, that indeed looks like a typo.
> ```
> store i16 0x5321, i16* %ptr
> ```
> would ofcourse put the 0x21 at the lower memory address. So this should say
>
> ```
> ; [%ptr + 0]: 00100001 (0x21)
> ; [%ptr + 1]: 01010011 (0x53)
> ```
@bjope should I submit the fix as NFC or will you do it?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94964/new/
https://reviews.llvm.org/D94964
More information about the llvm-commits
mailing list