search

JuliaHubOSS / llvm-cbe

resurrected LLVM "C Backend", with improvements
Other
826 stars 141 forks source link

Fix missing brace that breaks build for LLVM < 10.x #131

Closed makise-homura closed 3 years ago

makise-homura commented 3 years ago

Despite of the fact LLVM 9.x is unsupported anymore by llvm-cbe, it still probably works in most cases, if this brace is fixed.

I guess if there's a decision to forbid builds for any LLVM < 10.x, an #error should be there instead of such mistake. Still llvm-cbe seems to be working for me with LLVM 9.0.1, so I see no reason to put #error there instead of this fix.

Should fix issues #130 and #119.

makise-homura commented 3 years ago

Thanks!