Testbench

[n0thingNoob]

Summary
When data moves between PEs via links with register mapping, the generated
ASM instructions use incorrect direction operands. For example, PE(2,1) should
receive data in histogram from PE(2,2) via NORTH direction, but the original code generated
SOUTH instead.

Changes:

1. Fix handleRegisterRewiring: Add invertDir when computing incoming_dir
   for placeDstDeposit. The deposit instruction is placed on the destination
   tile and needs to know from which direction data arrives.

2. Fix handleDirectionRewiring: Calculate direction based on consumer's actual
   tile location instead of assuming it's always on the link destination tile.
   When consumer is on a different tile, compute the direction from destination
   tile to consumer tile.

Files Changed:

• lib/NeuraDialect/Transforms/GenerateCodePass.cpp
  • Line 519: Add invertDir for register deposit direction
  • Lines 535-564: Enhance handleDirectionRewiring to handle consumers on
    different tiles
  • Line 593: Pass topo parameter to handleDirectionRewiring

[Copilot]

Pull Request Overview

This PR improves the code generation pass for CGRA (Coarse-Grained Reconfigurable Array) routing by fixing direction computation for cross-tile data movement and standardizing variable naming conventions.

• Fixes direction rewiring logic to correctly compute incoming directions when consumers are not located on link destination tiles
• Adds explicit direction inversion for incoming directions at destination tiles in register deposits
• Standardizes parameter naming from camelCase to snake_case throughout affected functions

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.