pca006132
  • Joined on 2020-04-27
pca006132 deleted branch exception from M-Labs/nac3 2022-02-25 14:00:49 +08:00
pca006132 deleted branch rpc from M-Labs/nac3 2022-02-25 14:00:42 +08:00
pca006132 pushed to fix-exception at M-Labs/nac3 2022-02-25 14:00:15 +08:00
7998e0d8bd nac3core: fix exception final branch handling
pca006132 created branch fix-exception in M-Labs/nac3 2022-02-25 14:00:14 +08:00
pca006132 opened issue M-Labs/nac3#196 2022-02-22 20:38:24 +08:00
Fix exception handling for nested try/catch in loops
pca006132 commented on issue M-Labs/nac3#109 2022-02-22 17:23:05 +08:00
poor error message with tuple indexing

Fixed, this is due to a typo

pca006132 pushed to master at M-Labs/nac3 2022-02-22 17:22:32 +08:00
d3cb5d6e52 Fixed type error messages
pca006132 commented on issue M-Labs/nac3#166 2022-02-22 17:19:47 +08:00
confusing error message when attempting to append to list

Assuming by "diagnostics" you mean "figuring out compiler bugs".

No, I mean for users to figure out why the type is wrong... You need the generic parameters for more complicated cases.

pca006132 commented on issue M-Labs/nac3#166 2022-02-22 17:06:14 +08:00
confusing error message when attempting to append to list

Maybe make it optional? It then could be enabled with an environment variable that nac3artiq reads. Similar to ARTIQ_DUMP_LLVM etc. in the legacy compiler.

Well the problem is that, it may be…

pca006132 commented on issue M-Labs/nac3#166 2022-02-22 17:03:11 +08:00
confusing error message when attempting to append to list

We could remove the type variable, but in general the type variable would be useful for diagnostics for other cases. Making the type variable name deterministic is hard because the execution order is…

pca006132 commented on issue M-Labs/nac3#109 2022-02-22 17:01:37 +08:00
poor error message with tuple indexing

Well I guess this is due to constant folding and I forgot to add special case for that. Will fix.

pca006132 commented on pull request M-Labs/nac3#195 2022-02-22 14:34:29 +08:00
Better error messages

Just sort the strings then.

Done.

pca006132 pushed to error-messages at M-Labs/nac3 2022-02-22 14:33:57 +08:00
3ad25c8f07 nac3core: sort error messages for determinism
pca006132 commented on pull request M-Labs/nac3#195 2022-02-21 20:35:59 +08:00
Better error messages

Note that the error message order may not be deterministic.

What about sorting by (filename, line, column)?

We can, but this will need another refactor because we have to make every…

pca006132 commented on pull request M-Labs/nac3#195 2022-02-21 20:34:05 +08:00
Better error messages
`list[var72]::append` field does not exist at /home/pca006132/code/rust/nac3/nac3artiq/demo/demo.py: line 46 column 9

There is still this obscure "var72"...

Because it is…

pca006132 created pull request M-Labs/nac3#195 2022-02-21 18:43:18 +08:00
Better error messages
pca006132 pushed to error-messages at M-Labs/nac3 2022-02-21 18:42:06 +08:00
ede3706ca8 type_inferencer: special case tuple index error message
pca006132 pushed to error-messages at M-Labs/nac3 2022-02-21 18:28:16 +08:00
f97f93d92c applied rustfmt and clippy auto fix
d9cb506f6a nac3core: refactored for better error messages
352831b2ca nac3core: removed legacy location definition
Compare 3 commits »
pca006132 created branch error-messages in M-Labs/nac3 2022-02-21 18:28:15 +08:00
pca006132 commented on issue M-Labs/nac3#18 2022-02-18 15:12:46 +08:00
get rid of LLD

It's a bit silly, we'd still have to create temporary files and pass command line arguments into the library function call... maybe LLVM would accept a patch that improves the situation...