[PATCH] D48638: [Sparc] Add support for the cycle counter available in GR740

Daniel Cederman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 27 04:11:30 PDT 2018


dcederman added a comment.

In https://reviews.llvm.org/D48638#1213699, @jyknight wrote:

> LGTM.
>
> It's possibly only theoretical at the moment, since I don't know if anything both supports this and is little endian, but --
>  On a little endian system, will the registers be swapped; that is, would ASR22 hold the 32 LSB instead of ASR23?


We are not aware of any little endian system that supports this at the moment, but as the registers cannot be read together as a 64-bit value we do not think there is a reason to swap the registers for little endian systems.


https://reviews.llvm.org/D48638





More information about the llvm-commits mailing list