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

CodeGen/Generic/2003-05-28-ManyArgs.ll hits exponential behavior on PPC #1219

Closed
isanbard opened this issue Jul 26, 2006 · 8 comments
Closed
Assignees
Labels

Comments

@isanbard
Copy link
Contributor

Bugzilla Link 847
Resolution FIXED
Resolved on Feb 22, 2010 12:47
Version 1.6
OS MacOS X

Extended Description

The test/Regression/CodeGen/Generic/2003-05-28-ManyArgs.ll test-point was running for several
minutes on my machine before I killed it. I looked at it in the debugger and it seems to be doing the
legalizing of the selection DAG forever.

-bw

@isanbard
Copy link
Contributor Author

assigned to @lattner

@lattner
Copy link
Collaborator

lattner commented Jul 27, 2006

Verified, do you want me to take it?

@isanbard
Copy link
Contributor Author

I'd like to give it a shot. I'll look at it tonight.

@lattner
Copy link
Collaborator

lattner commented Jul 27, 2006

This looks like exponential behavior in LegalizeAllNodesNotLeadingTo, not an infinite loop. I'll see if it's
easy, if so I'd like to get it in before 1.8, so I'll fix it right away.

-Chris

@isanbard
Copy link
Contributor Author

Okay. I'll also look at it unless you close the bug. Yeah, it would be weird if it hung on one thing and not
everything.

@lattner
Copy link
Collaborator

lattner commented Jul 27, 2006

Have a fix. Sorry to steal this from you. :(

@isanbard
Copy link
Contributor Author

No biggie. :)

@lattner
Copy link
Collaborator

lattner commented Jul 27, 2006

Patch here:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20060724/036214.html

With this fix, CodeGen/Generic/2003-05-28-ManyArgs.ll compiles nearly instantly.

-Chris

@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
Labels
Projects
None yet
Development

No branches or pull requests

2 participants