[all-commits] [llvm/llvm-project] a84506: [AArch64] Use the same fast math preservation for ...

Craig Topper via All-commits all-commits at lists.llvm.org
Wed Nov 22 14:17:59 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a8450619358c3de23b72aef7c1fd932bca112257
      https://github.com/llvm/llvm-project/commit/a8450619358c3de23b72aef7c1fd932bca112257
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2023-11-22 (Wed, 22 Nov 2023)

  Changed paths:
    M llvm/lib/CodeGen/TargetInstrInfo.cpp
    M llvm/lib/Target/PowerPC/PPCInstrInfo.cpp
    M llvm/lib/Target/PowerPC/PPCInstrInfo.h
    M llvm/lib/Target/RISCV/RISCVInstrInfo.cpp
    M llvm/lib/Target/RISCV/RISCVInstrInfo.h
    M llvm/lib/Target/X86/X86InstrInfo.cpp
    M llvm/test/CodeGen/AArch64/machine-combiner-reassociate.mir
    A llvm/test/CodeGen/AArch64/pr72777.ll

  Log Message:
  -----------
  [AArch64] Use the same fast math preservation for MachineCombiner reassociation as X86/PowerPC/RISCV. (#72820)

Don't blindly copy the original flags from the pre-reassociated
instrutions.
This copied the integer poison flags which are not safe to preserve
after reassociation.
    
For the FP flags, I think we should only keep the intersection of
the flags. Override setSpecialOperandAttr to do this.

Fixes #72777.




More information about the All-commits mailing list