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

SSA Optimizations - SimpleCopyPropagate assertions #24

Closed
charsleysa opened this issue Mar 17, 2014 · 2 comments
Closed

SSA Optimizations - SimpleCopyPropagate assertions #24

charsleysa opened this issue Mar 17, 2014 · 2 comments
Labels
Feature
Milestone
1.x - Legacy

Comments

@charsleysa
Copy link
Member

SimpleCopyPropagate assertions happen due to IRInstruction.Phi be removed in this PR.

Is this a bug introduced by the PR or is this an existing bug?
@tgiphil
Copy link
Member

tgiphil commented Mar 18, 2014

It may be a combination of both; existing and new bugs.

What source code is causing the assertion? I would like to re-create it.

@charsleysa
Copy link
Member Author

If you're asking about the actual source code being compiled using the MOSA compiler then it is CoolWorld. I use CoolWorld as a good source for finding bugs.

If you're talking about the MOSA compiler code that throws the assertion, then that's the SimpleCopyPropagation method in SSAOptimizations stage in MOSA.Compiler.Framework.

charsleysa added a commit to charsleysa/MOSA-Project that referenced this issue Mar 19, 2014
@charsleysa
Fix and temporary workaround.
d0e4b18 
Fix for null operand issue.

Temporary work around for the following issues:
mosa#25
mosa#24
@charsleysa charsleysa mentioned this issue Mar 19, 2014
Merged
charsleysa added a commit to charsleysa/MOSA-Project that referenced this issue Mar 26, 2014
@charsleysa
Fix and temporary workaround.
a28bd3a 
Fix for null operand issue.

Temporary work around for the following issues:
mosa#25
mosa#24
@tgiphil tgiphil closed this as completed May 3, 2014
@tgiphil tgiphil added this to the 1.0 - Legacy milestone May 27, 2014
tgiphil added a commit that referenced this issue Jul 2, 2017
@tgiphil
Merge pull request #24 from robot9706/master
278ce20 
Connect and QEMUDebug dialogs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature
Projects
None yet
Milestone
1.x - Legacy
Development

No branches or pull requests
2 participants
@tgiphil @charsleysa