https://bugs.kde.org/show_bug.cgi?id=391311
Bug ID: 391311 Summary: [Patch] Fix arm64 valgrind tests compilation with clang Product: valgrind Version: 3.13.0 Platform: Other OS: Linux Status: UNCONFIRMED Severity: normal Priority: NOR Component: general Assignee: jsew...@acm.org Reporter: vtara...@google.com Target Milestone: --- Created attachment 111149 --> https://bugs.kde.org/attachment.cgi?id=111149&action=edit Fix arm64 valgrind tests compilation with clang Android Open Source Project compiles Valgrind using clang, with it's own assember, with semantics different from gcc's. Switching to external assember with "-no-integrated-as" is not an option, as it doesn't propagate arch feature expansions properly. The attached patch makes arm64 Valgrind tests compilation clang-friendly: - per https://developer.arm.com/docs/100069/0607/a64-floating-point-instructions/fcmp fcmp accept a floating point zero, replacing #0 with #0.0 - per http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.dui0802b/MOVI_advsimd_vector.html movi's 8-bit immediate is not allowed to be shifted, removing LSL #0. - ifdef-ing crypto assembly with __ARM_FEATURE_CRYPTO. Tests will be included in kryo, and excluded for generic arm64 builds. - per http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.dui0802b/ADD_addsub_ext.html add extended register's 64-bit version, extend is required to be uxtx -- You are receiving this mail because: You are watching all bug changes.