Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
[UR][Layer] Add Sanitizer Layer #1074
[UR][Layer] Add Sanitizer Layer #1074
Changes from 73 commits
4c9fe03
d6e589b
67fcd5e
1096b0f
c36bd72
6ba94ac
9ae8a99
9f90e43
336ce89
84a3afa
a8dadc1
4d63350
25b1183
e6bc306
23fb1b3
cb8a761
5c44b1d
a377c94
96eea7a
0a72ea9
8fa86dd
d9a51a7
730bf45
fed541a
ad2b1a7
dec5f32
ca58886
11cca13
3e6194e
89febf0
1b81df9
030400d
7208518
668a11e
f623135
7be8667
e4a47ac
61c0ca9
0ad1bb1
cc4402a
bf358c1
d5e7946
50666b7
de03096
b47ca29
1e2b6a0
42f6755
5aaba09
8dc90fd
3b819f6
54220e9
2328b48
e59d7c7
bba6f82
a084650
1148ccb
fa92f72
524a83d
e9a2093
a1c4ddc
71d8657
40b5ef6
0b00798
b29930b
c42bd9a
bc8cdb2
3bede09
5dc2d5c
30c0bc1
86d4056
49e2f4f
79998b9
6cc4913
577ddd3
4ee0e10
dc127d5
b59d931
28315db
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the names UR_ENABLE_SANITIZER and UR_USE_ASAN might be a little confusing in the sense that former is for device only and the latter is for host only. The difference isn't shown in the names.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I follow the same naming style as the tracing layer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the confusing still exists even if it is the same naming style.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree. IMO, the flag to enable layers needs a new naming style. @pbalcer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm,
UR_ENABLE_LAYER_XYZ
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this style is very similar to "UR_BUILD_ADAPTER_XYZ"