diff options
| author | Fernando Sahmkow <fsahmkow27@gmail.com> | 2019-09-23 08:15:31 -0400 |
|---|---|---|
| committer | FernandoS27 <fsahmkow27@gmail.com> | 2019-10-04 18:52:55 -0400 |
| commit | 7c756baa777cd27b319c1a397bd45270d2ffe041 (patch) | |
| tree | 4f0cdea5a803a61a91066c7d50ba6078f6ded866 /src/video_core/shader/ast.cpp | |
| parent | 5ea740beb52ee8ccbabef81397ce9458077c6a42 (diff) | |
Shader_IR: clean up AST handling and add documentation.
Diffstat (limited to 'src/video_core/shader/ast.cpp')
| -rw-r--r-- | src/video_core/shader/ast.cpp | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/video_core/shader/ast.cpp b/src/video_core/shader/ast.cpp index 54f0240e15..fc440526f1 100644 --- a/src/video_core/shader/ast.cpp +++ b/src/video_core/shader/ast.cpp @@ -432,6 +432,12 @@ void ASTManager::InsertReturn(Expr condition, bool kills) { program->nodes.PushBack(node); } +// The decompile algorithm is based on +// "Taming control flow: A structured approach to eliminating goto statements" +// by AM Erosa, LJ Hendren 1994. In general, the idea is to get gotos to be +// on the same structured level as the label which they jump to. This is done, +// through outward/inward movements and lifting. Once they are at the same +// level, you can enclose them in an "if" structure or a "do-while" structure. void ASTManager::Decompile() { auto it = gotos.begin(); while (it != gotos.end()) { @@ -656,11 +662,9 @@ void ASTManager::EncloseIfThen(ASTNode goto_node, ASTNode label) { sub_zipper->Init(goto_node, if_node); zipper.InsertAfter(if_node, prev); sub_zipper->Remove(goto_node); - // ShowCurrentState("After IfThen Enclose"); } void ASTManager::MoveOutward(ASTNode goto_node) { - // ShowCurrentState("Before MoveOutward"); ASTZipper& zipper = goto_node->GetManager(); const ASTNode parent = goto_node->GetParent(); ASTZipper& zipper2 = parent->GetManager(); |