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

Typo prevented visibility hiding from happening properly #1337

Merged
merged 1 commit into from
Feb 18, 2021

Conversation

lgritz
Copy link
Collaborator

@lgritz lgritz commented Feb 18, 2021

Fixes #1336

Signed-off-by: Larry Gritz [email protected]

@lgritz
Copy link
Collaborator Author

lgritz commented Feb 18, 2021

@johnhaddon

@johnhaddon
Copy link
Contributor

Thanks Larry, LGTM.

johnhaddon added a commit to johnhaddon/dependencies that referenced this pull request Feb 18, 2021
From AcademySoftwareFoundation/OpenShadingLanguage#1337.

This fixes conflicts with the system LLVM loaded by the Mesa driver.
johnhaddon added a commit to johnhaddon/dependencies that referenced this pull request Feb 18, 2021
From AcademySoftwareFoundation/OpenShadingLanguage#1337.

This fixes conflicts with the system LLVM loaded by the Mesa driver. It also means we can remove the Appleseed patch that was working around conflicts between liboslexec and liboslcomp. The first of many wrong trees I barked up until finding the problem in OSL.
@lgritz lgritz merged commit e5f26c0 into AcademySoftwareFoundation:master Feb 18, 2021
@lgritz lgritz deleted the lg-visibility branch February 18, 2021 16:44
lgritz added a commit to lgritz/OpenShadingLanguage that referenced this pull request Feb 18, 2021
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.

LLVM symbols not hidden in 1.11
2 participants