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

Entrega final #64

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

Entrega final #64

wants to merge 64 commits into from

Conversation

LuisLara-UH
Copy link

No description provided.

LuisLara-UH and others added 30 commits March 3, 2021 05:53
Major Changes:
Fill team information
Major Changes:
Add main.py
Add compiler components
… borrar zona de testing al final del archivo lexer.py)
Fix doge_generator ast issues
@jpconsuegra
Copy link
Collaborator

Evaluación

  • Luis Alejandro Lara Rojas
  • Carlos Alejandro Arrieta Montes de Oca

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

Comentarios

  • Vagos en la explicación de TODAS las fases.
  • Hacen SELF_TYPE pero no lo explican.
  • No explican concretamente cómo resuelven el case.
  • No explican cómo manejan los tipos por valor.

Nota final

Evaluación Nota
Final 2
Extraordinario 3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants