Skip to content

Conversation

@cyyever
Copy link
Contributor

@cyyever cyyever commented Dec 21, 2025

Apply more tidy fixes to gpu code.

@meta-cla meta-cla bot added the cla signed label Dec 21, 2025
@cyyever cyyever force-pushed the more_tidy9 branch 7 times, most recently from 95af750 to e66ebc0 Compare December 21, 2025 02:55
@q10
Copy link
Contributor

q10 commented Dec 22, 2025

Hi @cyyever can I trouble you to remove the structured bindings, since they cause compilation issues at least on internal builds with "capturing a structured binding is not yet supported in OpenMP" error?

@cyyever
Copy link
Contributor Author

cyyever commented Dec 23, 2025

@q10 I forgot they were removed by you.

Signed-off-by: Yuanyuan Chen <[email protected]>
@meta-codesync
Copy link
Contributor

meta-codesync bot commented Dec 23, 2025

@q10 has imported this pull request. If you are a Meta employee, you can view this in D89737616.

q10 pushed a commit to q10/FBGEMM that referenced this pull request Dec 24, 2025
Summary:
Apply more tidy fixes to gpu code.


Differential Revision: D89737616

Pulled By: q10
@q10
Copy link
Contributor

q10 commented Jan 2, 2026

@cyyever can I trouble you to remove the static keyword additions in this PR? They are preventing external linkage and causing symbol not found errors in our internal builds.

@cyyever cyyever force-pushed the more_tidy9 branch 2 times, most recently from 8d1370e to fa3729c Compare January 3, 2026 01:11
Signed-off-by: Yuanyuan Chen <[email protected]>
@cyyever
Copy link
Contributor Author

cyyever commented Jan 3, 2026

@q10 fixed

q10 pushed a commit to q10/FBGEMM that referenced this pull request Jan 5, 2026
Summary:
X-link: facebookresearch/FBGEMM#2277

Apply more tidy fixes to gpu code.


Differential Revision: D90046367

Pulled By: q10
@meta-codesync meta-codesync bot closed this in e13571c Jan 5, 2026
@meta-codesync
Copy link
Contributor

meta-codesync bot commented Jan 5, 2026

@q10 merged this pull request in e13571c.

@cyyever cyyever deleted the more_tidy9 branch January 6, 2026 00:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants