Skip to content

Weaken x86.get_pc_thunk for NDK <= r22#9994

Merged
yihonglyu merged 1 commit intomasterfrom
yilyu/weaken_x86.get_pc_thunk
Dec 10, 2021
Merged

Weaken x86.get_pc_thunk for NDK <= r22#9994
yihonglyu merged 1 commit intomasterfrom
yilyu/weaken_x86.get_pc_thunk

Conversation

@yihonglyu
Copy link
Copy Markdown
Contributor

@yihonglyu yihonglyu commented Dec 9, 2021

Otherwise multiple definition of '__x86.get_pc_thunk.bx' would cause error.

Otherwise there is multiple definition of '__x86.get_pc_thunk.bx'
Copy link
Copy Markdown
Contributor

@guoyu-wang guoyu-wang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please trigger a run on this pipeline https://aiinfra.visualstudio.com/Lotus/_build?definitionId=940&_a=summary and try compile with NDK 23 also
Thanks

@yihonglyu
Copy link
Copy Markdown
Contributor Author

Please trigger a run on this pipeline https://aiinfra.visualstudio.com/Lotus/_build?definitionId=940&_a=summary and try compile with NDK 23 also Thanks

https://aiinfra.visualstudio.com/Lotus/_build/results?buildId=186237&view=logs&j=1d5006dc-97a7-5b41-9d44-41bfffdb2d83 is passed. I compile ONNX Runtime locally with NDK 23 and it is passed as well.

@yihonglyu yihonglyu merged commit 3c79f30 into master Dec 10, 2021
@yihonglyu yihonglyu deleted the yilyu/weaken_x86.get_pc_thunk branch December 10, 2021 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants