[clang] [clang] [ARM] Explicitly enable NEON for Windows/Darwin targets (PR #122095)
David Spickett via cfe-commits
cfe-commits at lists.llvm.org
Wed Jan 8 05:41:53 PST 2025
DavidSpickett wrote:
Setting default features for a platform based on what we happened to choose (likely with a very thin justification) to be the default CPU sounds fragile to me, so I think this patch is a good step forward. Even if there weren't this downstream patch interacting with it, I think it would be a good change to make.
> This obviously is a downstream issue - but I'm trying to make it more explicit upstream, to make it easier for the downstreams that do patch it.
I wouldn't have expected the Debian patch to have these consequences either, so I agree with doing this.
https://github.com/llvm/llvm-project/pull/122095
More information about the cfe-commits
mailing list