Skip to content

Detect and error on failed codegen process#14762

Merged
straight-shoota merged 2 commits intocrystal-lang:masterfrom
ysbaddaden:fix/fork-codegen-error-stalls-compiler
Jul 1, 2024
Merged

Detect and error on failed codegen process#14762
straight-shoota merged 2 commits intocrystal-lang:masterfrom
ysbaddaden:fix/fork-codegen-error-stalls-compiler

Conversation

@ysbaddaden
Copy link
Collaborator

@ysbaddaden ysbaddaden commented Jun 27, 2024

Tries to report an exception when one occurs in a codegen forked process, otherwise detects when a codegen process terminated early (which is what happens on LLVM error). In both cases a BUG message is printed on stderr and the main process exits.

closes #14761

@ysbaddaden ysbaddaden added kind:bug A bug in the code. Does not apply to documentation, specs, etc. topic:compiler:codegen labels Jun 27, 2024
@ysbaddaden ysbaddaden self-assigned this Jun 27, 2024
@straight-shoota straight-shoota added this to the 1.13.0 milestone Jun 28, 2024
@straight-shoota straight-shoota changed the title Fix: detect failed codegen process Detect and error on failed codegen process Jul 1, 2024
@straight-shoota straight-shoota merged commit e279b3c into crystal-lang:master Jul 1, 2024
@ysbaddaden ysbaddaden deleted the fix/fork-codegen-error-stalls-compiler branch July 1, 2024 10:04
straight-shoota pushed a commit that referenced this pull request Jul 2, 2024
Report an exception when it occurs in a codegen forked process, otherwise detects when a codegen process terminated early (which is what happens on LLVM error). In both cases a BUG message is printed on stderr and the main process exits.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind:bug A bug in the code. Does not apply to documentation, specs, etc. topic:compiler:codegen

Projects

None yet

Development

Successfully merging this pull request may close these issues.

LLVM codegen error can stall the compiler

2 participants