saymrwulf/onnxruntime
1
0
Fork 0
mirror of https://github.com/saymrwulf/onnxruntime.git synced 2026-05-31 23:27:43 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix prefast scan complaints (#14823)

Browse source 
### Description

Fix prefast warning
https://dev.azure.com/aiinfra/ONNX%20Runtime/_workitems/edit/12821/


### Motivation and Context

This one is just excessive and annoying. It warned that I should cast
value to 8bytes before using * operator. But the expression is:
(size_t(64) * size_t(1024))

size_t is 8 bytes on 64b systems. And when it is 4 bytes on 32b systems,
the cast is expensive and completely unnecessary.
This commit is contained in:
Chen Fu 2023-02-24 17:23:22 -08:00 committed by GitHub
parent 6b83ad9659
commit bc5d0c83d1
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
onnxruntime/core/mlas/lib/mlasi.h
View file

@ -749,7 +749,7 @@ extern "C" {
#define MLAS_SGEMM_THREAD_COMPLEXITY (size_t(64) * size_t(1024))
#define MLAS_DGEMM_THREAD_COMPLEXITY (size_t(64) * size_t(1024))
#define MLAS_QGEMM_THREAD_COMPLEXITY (size_t(64) * size_t(1024))
#define MLAS_QGEMM_THREAD_COMPLEXITY 65536
//
// Single-threaded single precision matrix/matrix multiply operation.
@ -2076,7 +2076,7 @@ MlasMultiplyFloat64x2(MLAS_FLOAT64X2 Vector1, MLAS_FLOAT64X2 Vector2)
#endif
}
#endif
#endif // !MLAS_FLOAT64X2_UNSUPPORTED
//
// Reads a platform specific time stamp counter.