[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