[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