[all-commits] [llvm/llvm-project] 6112eb: [RISCV] Guard CFI emission code with MF.needsFrame...

[Sergei Barannikov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6112ebde0cdd31694536d0ac20a38e5f70f6185a
      https://github.com/llvm/llvm-project/commit/6112ebde0cdd31694536d0ac20a38e5f70f6185a
  Author: Sergei Barannikov <barannikov88 at gmail.com>
  Date:   2025-07-18 (Fri, 18 Jul 2025)

  Changed paths:
    M llvm/lib/Target/RISCV/RISCVFrameLowering.cpp
    M llvm/test/CodeGen/RISCV/short-forward-branch-opt.ll
    M llvm/test/CodeGen/RISCV/zdinx-spill.ll

  Log Message:
  -----------
  [RISCV] Guard CFI emission code with MF.needsFrameMoves() (#136060)

Currently, AsmPrinter skips CFI instructions created by a backend if
they are not needed. I'd like to change that so that it always
prints/encodes CFI instructions if a backend created them.

This change should slightly (perhaps negligibly) improve compile time as
post-PEI passes no longer need to skip over these instructions in
no-exceptions no-debug builds, and will allow to simplify convoluted
logic in AsmPrinter once other targets stop emitting CFI instructions
when they are not needed (that's my final goal).

The changes in a test seem to be caused by slightly different post-RA
scheduling in the absence of CFI instructions.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications