Pmux2shiftx

[stanminlee]

dont use newname, use newname2

[greptile-apps]

Greptile Summary

This PR replaces six NEW_ID calls (which generate file/line-based auto-indexed IDs) with NEW_ID2_SUFFIX(...) calls in pmux2shiftx.cc, giving the created wires and cells human-readable names derived from the parent $pmux cell name (e.g., <cell>_xor, <cell>_sub, <cell>_out). The cell variable from the outer loop is correctly in scope for all NEW_ID2_SUFFIX macro expansions, and module->uniquify() handles name deduplication across iterations.

Confidence Score: 5/5

Safe to merge — the change is a straightforward naming improvement with no logic or correctness risk.

All six substitutions use the correct macro (NEW_ID2_SUFFIX) that is well-defined for the current scope where cell is available, and module->uniquify() ensures no name collisions across loop iterations. No P0 or P1 issues found.

No files require special attention.

Important Files Changed

Filename	Overview
passes/opt/pmux2shiftx.cc	Six NEW_ID → NEW_ID2_SUFFIX replacements to produce descriptive names for generated wires/cells; macro is correctly in scope and uniquify handles deduplication.

_{Reviews (1): Last reviewed commit: "Merge branch 'main' of github.com:silima..." | Re-trigger Greptile}