Skip to content

is the finaaaal pull requeeeest, ninoninoooo #57

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
Open
Yumenio wants to merge 120 commits into
base: master
Choose a base branch
from
Open

is the finaaaal pull requeeeest, ninoninoooo #57

Yumenio wants to merge 120 commits into from

Conversation

@Yumenio
Copy link

@Yumenio Yumenio commented Mar 1, 2022

Pull request del equipo de
Richard García De la Osa C412
Andy A. Castañeda Guerra C412

Yumenio and others added 30 commits March 10, 2021 01:25
@Yumenio
Lexing and parsing, falta correr sendos tests
3ff8943
@Regnod
pasados tests de lexer, falta ver lo del estado del string
4e5b176
@Yumenio
agregando estado del string en lexer.py
f08d492
@Yumenio
merge del estado string con los errores del lexer, falta arreglar el …
ddf79a8 
…find_last
@Regnod
going to merge
bc7bff6
@Regnod
fixed wrong counting
077cb05
@Yumenio
Pasados todos los tests del parser
3ace997
@Regnod
salva
db30d2d
@Regnod
salva
e122e9a
@Regnod
pull exitoso
0ef3d53
@Regnod
añadiendo una lista de tokens a los nodods del ast
0bc825f
@Regnod
finishing semantics
05dafe9
@Regnod
adding cil ast and visitor with class declaration node done
879d3b6
@Yumenio
enganchando el visitor para traducir a CIL
8266445
@Regnod
merging
dd09029
@Regnod
trying to update
14ee851
@Regnod
merge some advances
0eba71f
@Yumenio
Merge parcial del codegen
d03bf98
@Yumenio
finished merging some codegen nodes
75bd9e5
@Yumenio
Some more nodes of codegen
549aa20
@Regnod
getting some changes
b10f93e
@Yumenio
getting back to work
8b8d8e9
@Regnod
...
b6cac38
@Regnod
...
b63f318
@Regnod
after merge
62f98bd
@Yumenio
implemented WhileNode, LesNode, LeqNode, EqualNode, StrinNode
423b079
@Yumenio
res
8ff4037
@Yumenio
correcting some nodes at cil_ast
91975fd
@Regnod
going to pull changes
bc2fac9
@Yumenio
push antes de los cambios de richard al cool_to_cil con el switch cas…
021cace 
…e implementado etc
Yumenio and others added 26 commits February 26, 2022 22:45
@jpconsuegra jpconsuegra self-assigned this Mar 12, 2022
@jpconsuegra
Copy link
Contributor

jpconsuegra commented Mar 14, 2022

Evaluación

  • Richard García De la Osa
  • Andy A. Castañeda Guerra

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

  • 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 4

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 Final Finalizado

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Yumenio @jpconsuegra @Regnod