Skip to content
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

Fix build on non-x86 non-ARM with clang #3671

Closed
wants to merge 1 commit into from
Closed

Fix build on non-x86 non-ARM with clang #3671

wants to merge 1 commit into from

Conversation

pkubaj
Copy link
Contributor

@pkubaj pkubaj commented Dec 2, 2021

Clang doesn't exit with error on powerpc64* after adding -mfpu=neon, but merely ignores the argument and prints a warning, which incorrectly makes it defined as ARM with NEON.

Clang doesn't exit with error on powerpc64* after adding -mfpu=neon, but merely ignores the argument and prints a warning, which incorrectly makes it defined as ARM with NEON.
@stweil
Copy link
Member

stweil commented Jan 1, 2022

@pkubaj, did you remove your tesseract repository? Meanwhile I could reproduce the issue and would like to merge your contribution.

Please set also your correct real user name for Git. It's currently set to 'pkubaj' in the commit message which does not look like your real name.

@stweil stweil added the bug label Jan 1, 2022
@stweil
Copy link
Member

stweil commented Jan 1, 2022

As soon as this issue is fixed, there remain more obstacles. See issue #3702 for more details.

@pkubaj
Copy link
Contributor Author

pkubaj commented Jan 6, 2022

Right, I removed it by mistake. It's recreated now.

@stweil
Copy link
Member

stweil commented Jan 6, 2022

Thank you. I could not merge it here but applied it in commit 790f3bd.

@stweil stweil closed this Jan 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants