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] Fix dereferencing nullptr #6763

Merged
merged 2 commits into from
Nov 30, 2022
Merged

Conversation

strongoier
Copy link
Contributor

Issue: fix #3323

Brief Summary

As #6759 allows the use of clang >= 12, we need to fix #3323 to pass the tests.

@netlify
Copy link

netlify bot commented Nov 29, 2022

Deploy Preview for docsite-preview canceled.

Name Link
🔨 Latest commit 9e046c9
🔍 Latest deploy log https://app.netlify.com/sites/docsite-preview/deploys/6385cbd862dd4e0008474645

Copy link
Contributor

@ailzhang ailzhang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ailzhang ailzhang merged commit 0abf1a7 into taichi-dev:master Nov 30, 2022
@strongoier strongoier deleted the fix-3323 branch November 30, 2022 03:06
strongoier added a commit to strongoier/taichi that referenced this pull request Nov 30, 2022
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
PENGUINLIONG pushed a commit that referenced this pull request Nov 30, 2022
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
quadpixels pushed a commit to quadpixels/taichi that referenced this pull request May 13, 2023
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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.

Release build with clang++12 failed some tests
2 participants