FreddyYe added a comment.

In D135937#3898228 <https://reviews.llvm.org/D135937#3898228>, @FreddyYe wrote:

> For saving capacity of ProcessorSubtypes, gcc decided to not support part of 
> compiler features of these two cpus:
>
>   __builtin_cpu_is("meteorlake")
>   __attribute__((target("arch=raptorlake")))
>   ... some others I don't know.
>
> Gcc's related patch:
> raptorlake 
> <https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=470a0659b508d684148f362c4dc0eccf5a83a23e>
>  and meteorlake 
> <https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=fd206f0e95fb6f41b96eaaaab1dc0c30378e5e08>
> Updated to align with gcc first. Welcome opinions and review!

My opinion is to aligin with gcc first: supported basic -march=xxx only for 
these two cpus and to support the features mentioned above in the future if 
needed. Moreover, these two cpus may have dedicated AdditionalTuning features 
in the future, we can also add at that time.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135937

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to