-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
oneTBB: tbb::atomic to std::atomic in sdf #2457
oneTBB: tbb::atomic to std::atomic in sdf #2457
Conversation
Filed as internal issue #USD-8375 |
I encountered two build issues related to this PR. I applied PRs #2457 through #2466 in order onto the current HEAD of the dev branch df0e5a5. The build was with
Full build output is https://gist.github.com/loqs/8a68505b5fcd7bc3443da60f224c2842 as it is too long to include.
Which I believe is related to OpenUSD/pxr/imaging/hd/dependencyForwardingSceneIndex.cpp Lines 214 to 220 in c912d78
There are more failures similar to this second failure which I believe all relate to OpenUSD/pxr/usdImaging/usdImaging/resolvedAttributeCache.h Lines 382 to 384 in c912d78
Changing the iterator to the none const version allows the build to succeed but if that was intended I would have expected it to be part of the PR. |
Thanks for testing. These errors seems to be due to a change to |
Thank you for the quick response. 65c0d86 indeed fixed it. |
Description of Change(s)
Swaps out tbb::atomic with std::atomic.
Split off from #1908
Fixes Issue(s)
Part of #1471, adding oneTBB support.