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.

Reply via email to