calyx icon indicating copy to clipboard operation
calyx copied to clipboard

Improve LSP error reporting

Open sgpthomas opened this issue 11 months ago • 3 comments

  • [x] Actually run papercut, well-formed, and synthesis-papercut passes
  • [x] support multiple errors in a single file
  • [x] figure out what to do about primitive files that have no main component
  • [x] Report parsing errors at reported location
  • [x] handle errors that don't report location
  • [x] add infrastructure for reporting warnings

sgpthomas avatar Mar 01 '24 17:03 sgpthomas

It's fun to flip through the error tests and see errors being reported.

https://github.com/calyxir/calyx/assets/3752122/949758e2-8a06-4e26-a1f5-9dd751d872af

sgpthomas avatar Mar 01 '24 17:03 sgpthomas

Dang, this is awesome.

sampsyo avatar Mar 02 '24 19:03 sampsyo

@sgpthomas should we merge this?

rachitnigam avatar Jul 03 '24 13:07 rachitnigam

Yeah

sgpthomas avatar Jul 04 '24 16:07 sgpthomas

Push the big green button when ready!

rachitnigam avatar Jul 05 '24 05:07 rachitnigam