[clang] [compiler-rt] [llvm] [X86] AMD Zen 5 Initial enablement (PR #107964)

Tobias Hieta via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 13 04:41:44 PDT 2024


tru wrote:

> The change to X86TargetParser.h looks ABI breaking to me.

This seems unfortunate to me. But I don't think it would be good to insert the enum at the end of the list and changing the sorting order.

How big of a problem would it be with a ABI break now? I know you have requested that we try to avoid those, even if it's strictly within the policy to still do that before the final release.

https://github.com/llvm/llvm-project/pull/107964


More information about the cfe-commits mailing list