Fix safeint-related errors in MLAS #26435
Open
+82
−82
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@lixing-star
Description
在 #26280 之前MLAS中没有使用到SafeInt,cmake脚本没有包含SafeInt头文件路径,编译会提示找不到SafeInt.hpp头文件,补丁中我添加了该路径(如果有其他更好方法请告诉我)在 #26280 中4cae30acd43efa4209fd4f8681932d7d3c3e9263 commit代码存在语法错误,SafeInt 3.0.28中并不存在Value()成员,编译器会报以下编译错误This patch has passed unit testing. Details are in LastTest.log.
该补丁通过了单元测试,详细信息在LastTest.logtest env:
LastTest.zip
Feel free to @ me for any future PRs related to loongarch—I'd be happy to assist with the review.
以后有loongarch相关PR欢迎@我,我很乐意协助审查