-
Notifications
You must be signed in to change notification settings - Fork 3.5k
feat: Add BFloat16 support for Gemm and MatMul CPU operators #26317
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
base: main
Are you sure you want to change the base?
Changes from 18 commits
4d31693
74a3353
d077b53
8157277
d25bbe5
e419576
e36ac3a
f76aca5
207726d
ef80220
5ecf919
49a5e52
ff426fb
640cdab
a915e40
9de02aa
1a52ca2
715518f
14ae0f1
36d23a7
0c02307
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -866,11 +866,19 @@ | |||||
InitializeCpuInfo(); | ||||||
} | ||||||
|
||||||
|
||||||
/* | ||||||
Discover all cores in a windows system. | ||||||
Note - every "id" here, given it be group id, core id, or logical processor id, starts from 0. | ||||||
*/ | ||||||
void WindowsEnv::InitializeCpuInfo() { | ||||||
// Initialize cpuinfo once on Windows similar to PosixEnv constructor. | ||||||
Comment on lines
873
to
+874
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we nee a macro here if cpuinfo supported? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I added code to force the library be available on Windows. |
||||||
(void)cpuinfo_initialize(); //Ignore the error if it failed to initialize | ||||||
|
(void)cpuinfo_initialize(); //Ignore the error if it failed to initialize | |
(void)cpuinfo_initialize(); // Ignore the error if it failed to initialize |
Check warning on line 877 in onnxruntime/core/platform/windows/env.cc
GitHub Actions / Optional Lint C++
[cpplint] reported by reviewdog 🐶
Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
Raw Output:
onnxruntime/core/platform/windows/env.cc:877: Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
Check warning on line 878 in onnxruntime/core/platform/windows/env.cc
GitHub Actions / Optional Lint C++
[cpplint] reported by reviewdog 🐶
Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
Raw Output:
onnxruntime/core/platform/windows/env.cc:878: Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
Check warning on line 880 in onnxruntime/core/platform/windows/env.cc
GitHub Actions / Optional Lint C++
[cpplint] reported by reviewdog 🐶
Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
Raw Output:
onnxruntime/core/platform/windows/env.cc:880: Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.