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

cool-compiler #54

Open
wants to merge 487 commits into
base: master
Choose a base branch
from
Open

cool-compiler #54

wants to merge 487 commits into from

Conversation

adrian13579
Copy link

No description provided.

rodrigo-pino and others added 30 commits February 22, 2022 11:17
@rodrigo-pino
Rewrite methods and attribute declaration transaltion to ccil, plus s…
f383e67 
…ome minor bug fixing
@rodrigo-pino
Mayor changes
e863078 
Remove Param, Local, and Arg instructions. They are now kept track by parent nodes accordingly.
Move helper function from ast module to visitor
Minor bug fixing
@rodrigo-pino
Delete old methods. They are now in ccil_gen.py
bd9b8bd
@rodrigo-pino
Minor bug fixing and refactoring
73d9a86
@adrian13579
Merge branch 'master' of https://github.com/CoolCows/cool-compiler-2021
96bff9d
@adrian13579
Update CCILToMIPSGenerator
06f3bc3
@rodrigo-pino
Minor refactoring in ccil visitor
61e786e
@rodrigo-pino
Add and redefine Atom CCIl nodes
04d8264
@rodrigo-pino
Add barebones for visiting constant values
a20e1e5
@rodrigo-pino
Fix and improve Call and Vcall
598bcdf
@rodrigo-pino
Remove helpers which turned out not to be so helpfuls
236eebc
@rodrigo-pino
Minor change in docs
ff01ddb
@adrian13579
Fix type annotations syntax
7ebefbe
@rodrigo-pino
Minor changes
cb226cb
@rodrigo-pino
Add more Operation as nodes (getAttr, setAttr, load, ...) and Data cl…
7b51225 
…ass to store values from .Data section
@rodrigo-pino
Add a simplified version of semantics scope, to track all cool variab…
91ee178 
…les and ccil counterparts across scopes
@rodrigo-pino
Mayor bug fixing in ccil visitor and some changes
16671c8 
Fix many bugs that always casused missmatch in user defined variables and and its ccil counter parts

Add track of ccil and cool names

Add new translations to Int String and Bool

Other minor bugs and refactoring
@rodrigo-pino
Add simple id(VariableNode) translation to ccil
eb490d9
@adrian13579
Remove ExpressionNode
7611835
@adrian13579
Handle case where node is None in class Node inicialization
82aa5de
@adrian13579
Merge branch 'master' of https://github.com/CoolCows/cool-compiler-2021
36217a7
@rodrigo-pino
fix(ccil_visitor): add param and attributes with old and new id inste…
ee2bf59 
…ad of new id and its type
@rodrigo-pino
Minor changes
1ee5873
@adrian13579
Update mips_gen.py
f0df4c7
@adrian13579
Fix function call in ccil_mips_gen.py
18a46d0
@adrian13579
Add syscall node in ast
2dfb13f
@adrian13579
Create __init__.py in code_gen
4621ca7
@adrian13579
Add push and pop methods in CCILToMIPSGenerator
90d156b
@adrian13579
Fix dynamic function call
9261a9f
@rodrigo-pino
Add new nodes and other changes
62010af
@jpconsuegra
Copy link
Collaborator

Evaluación

  • Rodrigo D. Pino Trueba
  • Adrián Rodríguez Portales

Requisitos mínimos (3)

  • El proyecto pasa todos los tests
  • El informe existe y explica el uso del compilador y la arquitectura básica

Requisitos intermedios (4)

  • Se implementaron las fases de chequeo semántico de forma separada y utilizando patrones adecuados
  • La arquitectura del compilador está organizada de forma coherente y mantenible

Requisitos extra (5)

  • Se implementaron las fases de generación de código intermedio y código objeto con patrones adecuados
  • Se resolvieron los problemas de representación de objetos en memoria e invocación de métodos virtuales de forma eficiente
  • Se resolvieron los problemas de boxing/unboxing de forma eficiente
  • Se resuelve el case de forma eficiente

Extra

  • Implementan SELF_TYPE
  • Implementan AUTO_TYPE

Comentarios

  • Siempre hacen boxing a los tipos por valor.
  • No resuelven el case eficientemente.

Nota final

Evaluación Nota
Final 2
Extraordinario 5

@jpconsuegra jpconsuegra self-assigned this Mar 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jpconsuegra jpconsuegra

Labels
Aprobado Extra Finalizado Suspenso Final
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@adrian13579 @jpconsuegra @apiad @rodrigo-pino