search

anza-xyz / move

Move compiler targeting llvm supported backends
https://discord.gg/wFgfjG9J
Apache License 2.0
109 stars 34 forks source link

Upgrade llvm-sys to 17.0.1 #435

Closed ksolana closed 6 months ago

ksolana commented 6 months ago

Fixes: #431

brson commented 6 months ago

Oof, I just merged @jcivlin's https://github.com/anza-xyz/move/pull/436, and it seems perhaps that broke tests here.

This seems good to me though.

ksolana commented 6 months ago

no worries i'll fix it forward.

jcivlin commented 6 months ago

Oof, I just merged @jcivlin's #436, and it seems perhaps that broke tests here.

This seems good to me though.

Lines in debug_info depend on llvm_sys versions. Both are correct and the diff is very light (so far), but since we do a literal comparison the tests fail. All is needed it the update of expected files. I would work on a filter in test comparison to make it less version dependent, but I do not know what will change or stay.