[PATCH] D147740: [llvm-profdata] Refactoring Sample Profile Reader to increase FDO build speed using MD5 as key to Sample Profile map

William Junda Huang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 13 12:18:33 PDT 2023


huangjd added a comment.

In D147740#4495250 <https://reviews.llvm.org/D147740#4495250>, @ro wrote:

> In D147740#4495119 <https://reviews.llvm.org/D147740#4495119>, @huangjd wrote:
>
>> Another question regarding ARM and SPARC, what is size_t on these machines? I suspect the error has something to do with that, since I couldn't replicate them on X64.
>
> Solaris `<iso/stddef_iso.h>` has
>
>   #if defined(_LP64) || defined(_I32LPx) 
>   typedef unsigned long   size_t;         /* size of something in bytes */
>   #else
>   typedef unsigned int    size_t;         /* (historical version) */ 
>   #endif
>
> Nothing unusual here.
>
> I still suspect that this is rather an endianess issue

I am waiting for the approval to use GCC farm machine so that I can debug it. Do you know any person I can ping to expedite the process?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147740



More information about the llvm-commits mailing list