craig.topper added a comment. In D111778#3064287 <https://reviews.llvm.org/D111778#3064287>, @FreddyYe wrote:
> In D111778#3063891 <https://reviews.llvm.org/D111778#3063891>, @erichkeane > wrote: > >> Supporting these types requires changes to the library so that we can check >> these, right? Do you have the library changes for these as well? The >> constants used to check for these features/etc needs to be present in the >> glibc library. > > Yes. It's highly related to the `ProcessorFeatures` I mentioned above. Does > the `library changes` refer to the compiler-rt/cpu_model.c? I think that > change depends on libgcc. For now libgcc has no strong will to update that > table. Every feature gcc knows about should be in libgcc. The unfortunate problem now is that cpu_features2 in libgcc is an array thats size may be different with every version of gcc. Clang can’t know how big that array is without reading the libgcc.a symbol table at compile time. Without knowing the size we can’t access any bits past the first 32 bits of cpu_features2. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111778/new/ https://reviews.llvm.org/D111778 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
