[CRL] Store winFlags in flagcxWindow to fix non-NVIDIA build failure

[MC952-arch]

PR Category

CRL

PR Types

Bug Fixes

PR Description

[Copilot]

Pull request overview

This PR fixes a non-NVIDIA build failure by persisting winFlags in the internal flagcxWindow object, avoiding reliance on vendor-specific flagcxInnerWindow fields that may not exist for non-NVIDIA adaptors.

Changes:

• Add winFlags to struct flagcxWindow and initialize it on the symmetric-heap (default) window path.
• Store winFlags on successful vendor window registration paths.
• Update flagcxDevMemCreate to derive symmetry from win->winFlags instead of win->vendorBase->winFlags.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
flagcx/flagcx.cc	Stores winFlags into flagcxWindow when vendor window registration succeeds (including DevComm staged windows).
flagcx/core/sym_heap.cc	Initializes w->winFlags for symmetric-heap (default) windows.
flagcx/core/include/sym_heap.h	Extends internal flagcxWindow definition with a winFlags field.
flagcx/adaptor/flagcx_device.cc	Uses win->winFlags to determine symmetric behavior for vendor windows, removing dependency on vendor window layout.

---

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

[yuejiewang]

LGTM

[mikethegoblin]

LGTM