[all-commits] [llvm/llvm-project] dc764f: [DirectX] initialize registers properties by calli...

Farzon Lotfi via All-commits all-commits at lists.llvm.org
Thu Feb 27 07:35:36 PST 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: dc764f5c689f5ee436b5835f8f8ccaea84317e03
      https://github.com/llvm/llvm-project/commit/dc764f5c689f5ee436b5835f8f8ccaea84317e03
  Author: Farzon Lotfi <farzonlotfi at microsoft.com>
  Date:   2025-02-27 (Thu, 27 Feb 2025)

  Changed paths:
    M llvm/lib/Target/DirectX/DirectXTargetMachine.cpp
    M llvm/unittests/Target/DirectX/CMakeLists.txt
    A llvm/unittests/Target/DirectX/RegisterCostTests.cpp

  Log Message:
  -----------
  [DirectX] initialize registers properties by calling addRegisterClass and computeRegisterProperties (#128818)

This fixes #126784 for the DirectX backend.
This bug was marked critical for DX so it is going to go in first. At
least one register class needs to be added via `addRegisterClass` for
`RegClassForVT` to be valid.
Further for costing information used by loop unroll and other
optimizations to be valid we need to call `computeRegisterProperties`.
This change does both of these.

The test cases confirm that we can fetch costing information off of
`getRegisterInfo` and that `DirectXTargetLowering` maps `i32` typed
registers to `DXILClassRegClass`.



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


More information about the All-commits mailing list