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.

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

tail merging badness with exception handling #1868

Closed
llvmbot opened this issue Jun 4, 2007 · 3 comments
Closed

tail merging badness with exception handling #1868

llvmbot opened this issue Jun 4, 2007 · 3 comments
Labels
bugzilla Issues migrated from bugzilla

Comments

@llvmbot
Copy link
Collaborator

llvmbot commented Jun 4, 2007

Bugzilla Link 1496
Resolution FIXED
Resolved on Nov 07, 2018 00:22
Version unspecified
OS Linux
Attachments testcase .ll
Reporter LLVM Bugzilla Contributor
CC @asl

Extended Description

If I compile the testcase like this
$ llc -enable-eh report.bc
then block invcont131 is removed (this
is the first block to be removed). It
should not be removed. If I turn off
tail merging then it is no longer removed.

Here is a valid path through the function:

entry -> bb -> cond_next44 -> cond_true80 ->
cond_next109 -> invcont131 -> cond_next618 (returns)
@llvmbot
Copy link
Collaborator Author

llvmbot commented Jun 4, 2007

I'll look.

@llvmbot
Copy link
Collaborator Author

llvmbot commented Jun 5, 2007

Indeed tail merging wasn't right for landing pads. Fixed.
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20070604/050215.html
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20070604/050217.html

@llvmbot
Copy link
Collaborator Author

llvmbot commented Jun 5, 2007

Closing per following comment (from private e-mail)
Yup, that fixed it - thanks a lot!

Best wishes,

Duncan.

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 3, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bugzilla Issues migrated from bugzilla
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@llvmbot