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

enable sanitizer in Arm64 build [skip ci] #1482

Merged
merged 1 commit into from
Oct 10, 2023

Conversation

GaryShen2008
Copy link
Collaborator

Close #1388

Signed-off-by: Gary Shen <[email protected]>
@GaryShen2008
Copy link
Collaborator Author

According to my test with the latest code, there's no failure to enable the sanitizer on Arm64 build.
So, just to enable it in the build script.

jlowe
jlowe previously approved these changes Oct 9, 2023
@jlowe
Copy link
Member

jlowe commented Oct 9, 2023

build

@GaryShen2008 GaryShen2008 changed the title enable sanitizer inn Arm64 build enable sanitizer in Arm64 build Oct 9, 2023
@GaryShen2008
Copy link
Collaborator Author

build

@pxLi pxLi added bug Something isn't working build labels Oct 10, 2023
@pxLi pxLi changed the base branch from branch-23.10 to branch-23.12 October 10, 2023 00:54
@pxLi pxLi dismissed jlowe’s stale review October 10, 2023 00:54

The base branch was changed.

@pxLi pxLi changed the title enable sanitizer in Arm64 build enable sanitizer in Arm64 build [skip ci] Oct 10, 2023
@pxLi
Copy link
Collaborator

pxLi commented Oct 10, 2023

skip ci as no arm CI for pre-merge currently

@pxLi
Copy link
Collaborator

pxLi commented Oct 10, 2023

build

@pxLi pxLi merged commit 6dd74fc into NVIDIA:branch-23.12 Oct 10, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Sanitizer failed in arm64 test
3 participants