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

NoBugsNoLife compiler implementation #34

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

Conversation

karlaoh99
Copy link

Lexer, parser and semantic logic are added

Vitico99 and others added 30 commits April 17, 2021 04:28
Added required interface method so it can be used by the ply yacc parser: input(str) and
token()

Fix small bug with comment line messing with the token location
Add lexer, parser and type ckecker logic
@jpconsuegra jpconsuegra self-requested a review March 8, 2022 10:56
@jpconsuegra
Copy link
Collaborator

jpconsuegra commented Mar 8, 2022

Evaluación

  • Amanda González Borrell
  • Karla Olivera Hernández
  • Victor Manuel Cardentey Fundora

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 resolvió el case eficientemente.

Comentarios

  • Siempre hacen boxing. SOLVED
  • Resuelven el case generando todos los descendientes. SOLVED

Nota final

Evaluación Nota
Final 4
Extra 5

@jpconsuegra
Copy link
Collaborator

Revisado

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.

4 participants