-
Notifications
You must be signed in to change notification settings - Fork 700
Add tidy fixes #5268
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add tidy fixes #5268
Conversation
95af750 to
e66ebc0
Compare
|
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? |
|
@q10 I forgot they were removed by you. |
Signed-off-by: Yuanyuan Chen <[email protected]>
Summary: Apply more tidy fixes to gpu code. Differential Revision: D89737616 Pulled By: q10
|
@cyyever can I trouble you to remove the |
8d1370e to
fa3729c
Compare
Signed-off-by: Yuanyuan Chen <[email protected]>
|
@q10 fixed |
Summary: X-link: facebookresearch/FBGEMM#2277 Apply more tidy fixes to gpu code. Differential Revision: D90046367 Pulled By: q10
Apply more tidy fixes to gpu code.