[go: up one dir, main page]
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.

Sign up for GitHub

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

Jump to bottom

Deferred diagnostics are broken for In-class initializers #1032

Open
Fznamznon opened this issue Jan 21, 2020 · 2 comments
Open

Deferred diagnostics are broken for In-class initializers #1032

Fznamznon opened this issue Jan 21, 2020 · 2 comments
Labels
bug Something isn't working confirmed

Comments

@Fznamznon
Copy link
Contributor
Fznamznon commented Jan 21, 2020 •
edited
Loading

The following test case cannot be diagnosed with deferred diagnostics system:

int foo() { throw 1; }
struct S {
    int bar = foo();
};
SYCL_EXTERNAL void bar() { S s; }

We do our reporting at a functiondecl->caller level, when the declcontext can actually be a CXXRecordDecl as well.
NOTE: This seems broken not only for SYCL, but for any other programming model which allows in-class initializes.
@Fznamznon Fznamznon self-assigned this Jan 21, 2020
@bader bader added the bug Something isn't working label Jan 21, 2020
@Fznamznon Fznamznon mentioned this issue Jan 21, 2020
Merged
@Fznamznon Fznamznon removed their assignment Feb 11, 2020
@github-actions
Copy link
Contributor

This issue is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be automatically closed in 30 days.

@github-actions github-actions bot added the Stale label Feb 19, 2022
@Fznamznon
Copy link
Contributor Author

The issue is still here.

@Fznamznon Fznamznon removed the Stale label Feb 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working confirmed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@bader @alexbatashev @Fznamznon