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

[BUG]Invalid code in SdfChildrenProxy fails to compile with clang19 #3442

Open
mont29 opened this issue Dec 2, 2024 · 5 comments
Open

[BUG]Invalid code in SdfChildrenProxy fails to compile with clang19 #3442

mont29 opened this issue Dec 2, 2024 · 5 comments

Comments

@mont29
Copy link

mont29 commented Dec 2, 2024

Description of Issue

Trying to build a project using OpenUSD with Clang19 fails with following error:

[...]/lib/linux_x64/usd/include/pxr/usd/sdf/childrenProxy.h:75:21: error: no member named '_Set' in 'SdfChildrenProxy<_View>'
   75 |             _owner->_Set(*_pos, x);
      |             ~~~~~~  ^

Issue discovered when trying to build Blender with Clang 19.

This is related to this change in the compiler (see also the PR).

System Information (OS, Hardware)

Operating System: Linux Debian Testing, x86_64
Other: Clang-19

Package Versions

Version / Commit SHA: USD 24.05

Issue seems to still be present in current main, code looks unchanged (although affected line is now pxr/usd/sdf/childrenProxy.h:75).

@nvmkuruc
Copy link
Collaborator

nvmkuruc commented Dec 2, 2024

I think @dgovil's change #3434 will address this.

@dgovil
Copy link
Contributor

dgovil commented Dec 2, 2024

Thanks for those links, Bastien. Yeah, this is very timely as I just hit this last week and I believe my changes should resolve it for you. Please let me know if you hit anything else, as I've only tested against our own Apple compiler forks.

@mont29
Copy link
Author

mont29 commented Dec 2, 2024

@dgovil that was the only issue I had with USD libraries on a full Blender build, so indeed your patch should cover it all. :)
FYI the only other similar issue we have in Blender so far is with OpenVDB.

@dgovil
Copy link
Contributor

dgovil commented Dec 2, 2024

Ah that's really good to know about OpenVDB.

@jesschimein
Copy link
Collaborator

Filed as internal issue #USD-10484

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

No branches or pull requests

4 participants