On Tue, 2019-04-02 at 21:30 +0200, Peter Kjellerstedt wrote: > These patches are intended to improve the ARM tunings and came about > after I had to study the tune files a lot more than I first had > anticipated... > > The first patch (arch-armv8a.inc: Correct > PACKAGE_EXTRA_ARCHS_tune-armv8a-*) avoids odd architectures such as > "crc" and "crypto" to be included in PACKAGE_EXTRA_ARCHS. > > The following three patches (where i have sent the latter two to the > list before) restores the "armv*" features to TUNE_FEATURES for all ARM > based SoCs as it does not make sense to remove them from the SoC > specific tune files just to determine whether -mcpu or -march shall be > used. This is because the SoCs still support the armv* features > specified in their TUNE_FEATURES even if they also have more specific > SoC tunings.
It depends how you view TUNE_FEATURES and what the presence of things there is defined to mean. "armv4t" is defined in the arm tune files to mean "add -march=armv4t" which is the convention used throughout all the tune files. I'm still not convinced this part of your changes improves the situation as it then breaks and confuses that situation. I'm also worried you're wanting this as you have code elsewhere which is using "armvX" there to mean something else? Cheers, Richard -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core