Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ExceptionHandling] Raise error at the exact failed lowering pass #139

Open
zzzDavid opened this issue Sep 8, 2022 · 0 comments
Open

[ExceptionHandling] Raise error at the exact failed lowering pass #139

zzzDavid opened this issue Sep 8, 2022 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@zzzDavid
Copy link
Collaborator

zzzDavid commented Sep 8, 2022

The LLVM backend goes through a series of lowering passes:

        hcl_d.lower_composite_type(module)
        hcl_d.lower_fixed_to_int(module)
        hcl_d.lower_print_ops(module)
        hcl_d.lower_anywidth_int(module)

However, when any pass failed, the error is raised at execution engine creation. Often times we get an error saying

RuntimeError: Failure while creating ExecutionEngine

But it's one of the pass that failed. I think this is pretty confusing and we should improve error raising (at the C++ side) to raise the error right after the failed pass.

@zzzDavid zzzDavid self-assigned this Sep 8, 2022
@zzzDavid zzzDavid added the enhancement New feature or request label Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant