-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
compiler: Implement @disableIntrinsics()
builtin function.
#22154
base: master
Are you sure you want to change the base?
Conversation
Might be nice to add docs for this in the langref as well, along with disableInstrumentation maybe? (#21424)
Or I guess it can be done then |
Will write docs if/when it's approved. |
45ffc0f
to
5fe1fe6
Compare
b9a6861
to
c45775b
Compare
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.
Looks good other than this strange snippet I see here
c45775b
to
e11ac02
Compare
@andrewrk do you have any thoughts on nuking |
Yeah I'm in favor |
you have the patience of a saint |
👼 |
This implements #22110, and in a way that also satisfies the use case for #21833. If accepted, this would therefore supersede #21900.
I won't add any usage of this builtin to compiler-rt and std yet as this will require a
zig1.wasm
update.Closes #21833.
Closes #22110.