-
Notifications
You must be signed in to change notification settings - Fork 365
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
Allow running of erased macros #5744
Conversation
Please add some test cases showing that one cannot use this new feature to turn erased values into run-time values. Some ideas (untested):
|
Here are the results of trying your test cases:
|
I've fixed the error location and added support for using |
The only failed test in CI is because of this error:
I assume this is unrelated so I am going to merge this PR now. |
As discussed in #5734, macros are always executed during compile time so it should be allowed to mark them as erased.