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

build: add options to use static Cuda libraries #1772

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

lgritz
Copy link
Collaborator

@lgritz lgritz commented Feb 20, 2024

  • New build option CUDA_PREFER_STATIC_LIBS tries to use Cuda's static libraries if they can be found.
  • Remove code related to nvrt library, it's no longer needed in OptiX 7.
  • Changes in checked_find_package to allow a PREFER_STATIC_LIBS optional argument. Turns out we didn't use this, but I think it's useful functionality to keep now that I've written it.

* New build option `CUDA_PREFER_STATIC_LIBS` tries to use Cuda's static
  libraries if they can be found.
* Remove code related to nvrt library, it's no longer needed in OptiX 7.
* Changes in checked_find_package to allow a PREFER_STATIC_LIBS
  optional argument. Turns out we didn't use this, but I think it's
  useful functionality to keep now that I've written it.

Signed-off-by: Larry Gritz <[email protected]>
@lgritz lgritz requested a review from chellmuth February 20, 2024 22:11
@lgritz lgritz merged commit 77ceebb into AcademySoftwareFoundation:main Feb 21, 2024
23 checks passed
@lgritz lgritz deleted the lg-cudastatic branch February 21, 2024 01:13
lgritz added a commit to lgritz/OpenShadingLanguage that referenced this pull request Feb 21, 2024
…ation#1772)

* New build option `CUDA_PREFER_STATIC_LIBS` tries to use Cuda's static
  libraries if they can be found.
* Remove code related to nvrt library, it's no longer needed in OptiX 7.
* Changes in checked_find_package to allow a PREFER_STATIC_LIBS
  optional argument. Turns out we didn't use this, but I think it's
  useful functionality to keep now that I've written it.

Signed-off-by: Larry Gritz <[email protected]>
chellmuth pushed a commit to chellmuth/OpenShadingLanguage that referenced this pull request Sep 6, 2024
This won't be picked up by the platforms yet, they are still using
1.13.5.x for 2023.10 (as well as still OIIO 2.6.0.x, which is too old
for this OSL release). So really the only client for this build is
likely Arnold.

Highlights:

* Raise OpenImageIO to 2.6.1.0.
* RS::get_texture_info make robust to empty shaderglobals param (AcademySoftwareFoundation#1731)
* LLVM opaque pointers support (AcademySoftwareFoundation#1728)
* LLVM new pass manager support (AcademySoftwareFoundation#1729)
* LLVM 16 and 17 support (AcademySoftwareFoundation#1730)
* fix: SymOverrideInfo bitfields should be the same type (AcademySoftwareFoundation#1745)
* fix: batched pointcloud if "index" is not passed (AcademySoftwareFoundation#1753)
* api: Ustringhash phase 3 (AcademySoftwareFoundation#1732)
* fix: Fix NVPTX TargetMachine leak, etc. (AcademySoftwareFoundation#1763)
* feat(api): Add API for building interpolated getter free functions. (AcademySoftwareFoundation#1765)
* build: add options to use static Cuda libraries (AcademySoftwareFoundation#1772)
* build(deps): Initial support for LLVM-18 (AcademySoftwareFoundation#1773)
* fix: Mute partio error prints (AcademySoftwareFoundation#1774)
* fix: calculatenormal needs fliphandedness (AcademySoftwareFoundation#1783)
* fix: Print closure missing error message at compile time instead of run time.  (AcademySoftwareFoundation#1781)
* fix: GPU interpolated param initialization (AcademySoftwareFoundation#1791)
* oslquery: Simplify include and link needs (AcademySoftwareFoundation#1787)
* Make isconnected() work with downstream renderer "connections." (AcademySoftwareFoundation#1782)

Also along for the ride:

* Remove the `blah-platform/=1.0` preamble, no longer needed.
* Stop building for deprecated Nuke 12 and 13 platforms.
* Raise cmake build version to 3.24

See merge request spi/dev/3rd-party/osl-feedstock!57
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.

2 participants