[all-commits] [llvm/llvm-project] f57fb8: [FPEnv][AArch64] Correct strictfp tests.

Kevin P. Neal via All-commits all-commits at lists.llvm.org
Wed Jul 26 06:14:50 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f57fb82e0f90b4c667a0d044c14d9deaa22d3a99
      https://github.com/llvm/llvm-project/commit/f57fb82e0f90b4c667a0d044c14d9deaa22d3a99
  Author: Kevin P. Neal <kevin.neal at sas.com>
  Date:   2023-07-26 (Wed, 26 Jul 2023)

  Changed paths:
    M llvm/test/CodeGen/AArch64/arm64-constrained-fcmp-no-nans-opt.ll
    M llvm/test/CodeGen/AArch64/arm64-fmadd.ll
    M llvm/test/CodeGen/AArch64/arm64-vmul.ll
    M llvm/test/CodeGen/AArch64/neon-fpextend_f16.ll
    M llvm/test/CodeGen/AArch64/neon-scalar-by-elem-fma.ll
    M llvm/test/CodeGen/AArch64/strict-fp-opt.ll

  Log Message:
  -----------
  [FPEnv][AArch64] Correct strictfp tests.

Correct AArch64 strictfp tests to follow the rules documented in the LangRef:
https://llvm.org/docs/LangRef.html#constrained-floating-point-intrinsics

Mostly these tests just needed the strictfp attribute on function
definitions.  I've also removed the strictfp attribute from uses
of the constrained intrinsics because it comes by default since
D154991, but I only did this in tests I was changing anyway.

I have removed attributes added to declare lines of intrinsics. The
attributes of intrinsics cannot be changed in a test so I eliminated
attempts to do so.

Test changes verified with D146845.




More information about the All-commits mailing list