search

PLC-lang / rusty

Structured Text Parser and LLVM Frontend
GNU Lesser General Public License v3.0
181 stars 47 forks source link

Move variables to ELF sections with mangled names #1198

Open CohenArthur opened 1 month ago

CohenArthur commented 1 month ago

Add the same logic as #1160 but for global variables

CohenArthur commented 1 month ago

The CI test failure is extremely weird! 

fatal runtime error: failed to initiate panic, error 5
error: test failed, to rerun pass `-p iec61131std --test string_function_tests`

I can't reproduce it locally when running the workflow's command (./scripts/build.sh --test). I think it might just be a github actions blurp and it just needs a rerun?

volsa commented 1 month ago

I can't reproduce it locally when running the workflow's command (./scripts/build.sh --test). I think it might just be a github actions blurp and it just needs a rerun?

Yup, for some reason this happens occasionally. I've triggered a re-run which should hopefully fix the issue.

CohenArthur commented 1 month ago

thanks @volsa ! it did :D