topperc wrote: > I think we should reject `-mtune=native:<tunefeatures>`. It wouldn't be > portable - as soon as you compile on a different machine you may get errors > because those tune features are not supported. At least > `-mtune=<cpu>:<tunefeatures>` will always give the same errors. > > It's probably the case that `-mtune=native` should expand to > `-mtune=<cpu>:<tunefeatures>` anyway, so the `native` expansion happens > before tunefeature parsing.
That was the previous behavior so the patch is NFC now. It's just structured better and includes a comment and test. https://github.com/llvm/llvm-project/pull/215891 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
