Skip to content
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.

Sign up for GitHub

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

Jump to bottom

JIT: reconsider use of BBF_DONT_REMOVE on EH blocks #82336

Open
BruceForstall opened this issue Feb 18, 2023 · 1 comment
Open

JIT: reconsider use of BBF_DONT_REMOVE on EH blocks #82336

BruceForstall opened this issue Feb 18, 2023 · 1 comment
Assignees
@BruceForstall
Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI
Milestone
Future

Comments

@BruceForstall
Copy link
Member

try/handler begin/last blocks are marked with BBF_DONT_REMOVE. Do we still need them to be? Can we check the block EH index instead when necessary to determine the EH status of a block? Can we generalize flow updates?
@BruceForstall BruceForstall added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Feb 18, 2023
@BruceForstall BruceForstall added this to the 8.0.0 milestone Feb 18, 2023
@BruceForstall BruceForstall self-assigned this Feb 18, 2023
@ghost
Copy link

ghost commented Feb 18, 2023

Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch, @kunalspathak
See info in area-owners.md if you want to be subscribed.

Issue Details

try/handler begin/last blocks are marked with BBF_DONT_REMOVE. Do we still need them to be? Can we check the block EH index instead when necessary to determine the EH status of a block? Can we generalize flow updates?

Author: BruceForstall
Assignees: BruceForstall
Labels:

area-CodeGen-coreclr
Milestone: 8.0.0

@BruceForstall BruceForstall modified the milestones: 8.0.0, Future Jun 22, 2023
@BruceForstall BruceForstall mentioned this issue Oct 12, 2023
Merged
@BruceForstall BruceForstall mentioned this issue Nov 9, 2023
Merged
@BruceForstall BruceForstall mentioned this issue Sep 20, 2024
Merged
@BruceForstall BruceForstall mentioned this issue Sep 30, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BruceForstall BruceForstall

Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI
Projects
None yet
Milestone
Future
Development

No branches or pull requests
1 participant
@BruceForstall