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

Entrega final proyecto de compilacion #53

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

Entrega final proyecto de compilacion #53

wants to merge 6 commits into from

Conversation

abelo98
Copy link

@abelo98 abelo98 commented Feb 26, 2022

No description provided.

@jpconsuegra jpconsuegra self-requested a review March 8, 2022 14:37
@jpconsuegra jpconsuegra self-assigned this Mar 8, 2022
@jpconsuegra
Copy link
Contributor

jpconsuegra commented Mar 8, 2022 •
edited
Loading

Evaluación

  • Abel Antonio Cruz Suarez
  • Daniel Reynel Dominguez Ceballos

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

  • Siempre hacen boxing.
  • No dan muchos detalles respecto a cómo solucionan el case. [SOLVED]

Nota final

Evaluación Nota
Final 4
Extraordinario 5

@jpconsuegra
Copy link
Contributor

Revisado

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jpconsuegra jpconsuegra Awaiting requested review from jpconsuegra

Assignees

@jpconsuegra jpconsuegra

Labels
Aprobado Final Finalizado
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@abelo98 @jpconsuegra