-
Notifications
You must be signed in to change notification settings - Fork 164
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
Warnings target switch intrinsic asm #4727
Warnings target switch intrinsic asm #4727
Conversation
7e373a1
to
5f3eb0f
Compare
…witch-intrinsic-asm
@@ -16,23 +16,8 @@ struct ReachabilityContext | |||
ReachabilityContext() = default; | |||
ReachabilityContext(IRGlobalValueWithCode* code); | |||
|
|||
bool isInstReachable(IRInst* from, IRInst* to); |
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.
We can revert this change and move isInstReachable
to its original place?
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.
We can but after switching to getBlock
calls we would also need to #include "slang-ir-util.h"
in slang-ir-reachabilty.h
is that fine?
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.
Ah, I see, moving it here is fine.
Handles #4720
Also adds a fix to skip generating IR for empty target switches. Previously this would result in generating an unreachable block, which would have necessitated special handling for target switches.