Skip to content

Comments

For all assembler scripts where it matters, recognise clang > 9.x#10855

Closed
levitte wants to merge 1 commit intoopenssl:masterfrom
levitte:fix-10853
Closed

For all assembler scripts where it matters, recognise clang > 9.x#10855
levitte wants to merge 1 commit intoopenssl:masterfrom
levitte:fix-10853

Conversation

@levitte
Copy link
Member

@levitte levitte commented Jan 15, 2020

Fixes #10853

@levitte levitte added branch: master Applies to master branch approval: review pending This pull request needs review by a committer branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL) labels Jan 15, 2020
@levitte levitte removed the branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL) label Jan 15, 2020
@levitte
Copy link
Member Author

levitte commented Jan 15, 2020

Because the reporter was using 1.1.1, I'll make a separate PR for that branch, to make testing easier

@paulidale paulidale added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Jan 15, 2020
openssl-machine pushed a commit that referenced this pull request Jan 17, 2020
@levitte
Copy link
Member Author

levitte commented Jan 17, 2020

Merged.

9bb3e5f For all assembler scripts where it matters, recognise clang > 9.x

@levitte levitte closed this Jan 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals branch: master Applies to master branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

libcrypto: incorrect compiler version check in perl script crypto/bn/asm/x86_64-mont5.pl

2 participants