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

casos local aceptados #44

Closed
wants to merge 8 commits into from
Closed

casos local aceptados #44

wants to merge 8 commits into from

Conversation

tingui96
Copy link

todos los casos en la pc aceptados, falta terminar informe y el visual

Copy link
Author

@tingui96 tingui96 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Casos locales aceptados

Copy link

@Juano97 Juano97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tratando de correr test en GH

@tingui96 tingui96 closed this Feb 25, 2022
@tingui96 tingui96 reopened this Feb 25, 2022
@jpconsuegra jpconsuegra self-assigned this Mar 10, 2022
@jpconsuegra
Copy link
Contributor

Evaluación

  • Yandy Sanchez Orosa
  • Juan Carlos Vazquez Garcia

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

Comentarios

  • No encontré en el informe cómo se organiza el proyecto en módulos.
  • No encontré en el informe muchos detalles sobre cómo se divide el chequeo semántico.
  • No encontré de forma clara en el informe cómo se organiza la memoria en runtime.
  • No encontré en el informe cómo se resuelven los métodos virtuales de forma que la invocación polimórfica sea O(1).
  • No encontré en el informe cómo resuelven los problemas de boxing/unboxing.
  • Resuelven el case generando todos los descendientes.

Nota final

Evaluación Nota
Final 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.

3 participants