-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
detect forbidden aliasing #1263
Conversation
As previously mentioned in the original issue(#177), there might be a lot of other candidate functions that need the same enhancement. If you confirm the current pattern, I can apply it to them as well. |
I resolved a merge conflict in |
This is a squash of some existing work from Arzaghi, plus a bunch of work from nimazzuc Co-authored-by: Hamid Reza Arzaghi <arzaghi@outlook.com>
65f0ab7
to
71852bb
Compare
I've pushed a cleanup and merge commit; this squashes the existing work, since the last "real" commit was from two years ago. |
There are numerous test failures that need to be investigated. |
Thanks! (And apologies for taking 2 years to review this! 🙀 And thanks @strega-nil-ms for cleaning up the accumulated merge conflicts! 😻) I've pushed some changes, including a major expansion of test coverage:
I explored applying the 4-arg overload to |
I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed. |
Thanks again for improving these debugging checks! 🐞 🚀 🎉 |
Fixes #177