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

ROOT produces incorrect dictionary for constrained class templates #13461

Open
1 task done
greenc-FNAL opened this issue Aug 15, 2023 · 8 comments
Open
1 task done

ROOT produces incorrect dictionary for constrained class templates #13461

greenc-FNAL opened this issue Aug 15, 2023 · 8 comments

Comments

@greenc-FNAL
Copy link
Collaborator

Check duplicate issues.

  • Checked for duplicates

Description

The attached reproducer will fail to execute the ROOT Python script successfully when the buggy::Constraint class template is constrained instead of producing identical output to when buggy::Constraint is not constrianed:

$ ./makeit good
number = 5
$ ./makeit
Error in <TInterpreter::AutoParse>: Error parsing payload code for class buggy::Constrained with content:
⋮

Reproducer

To induce the failure case:

$ cd root-concepts-dict-bug
$ ./makeit

To see the reference output:

$ cd root-concepts-dict-bug
$ ./makeit good

ROOT version

ROOT 6.28/04@30598a8430

Installation method

UPS

Operating system

SL7

Additional context

Compiled with GCC 13.1.0/C++20.

@greenc-FNAL
Copy link
Collaborator Author

@greenc-FNAL
Copy link
Collaborator Author

Trying to build the head of v6-28-00-patches (currently 281a162) to reproduce against.

@greenc-FNAL
Copy link
Collaborator Author

Trying to build the head of v6-28-00-patches (currently 281a162) to reproduce against.

Does not build currently...

@jalopezg-git
Copy link
Collaborator

Trying to build the head of v6-28-00-patches (currently 281a162) to reproduce against.

Does not build currently...

@greenc-FNAL Thanks for reporting this and providing a reproducer! Any chance that you can try with ROOT from the master branch too?

@pcanal
Copy link
Member

pcanal commented Aug 15, 2023

There is no reason for this to be fixed in the master (since LLVM version is not changed yet)

@greenc-FNAL
Copy link
Collaborator Author

Trying to build the head of v6-28-00-patches (currently 281a162) to reproduce against.

Does not build currently...

@greenc-FNAL Thanks for reporting this and providing a reproducer! Any chance that you can try with ROOT from the master branch too?

Might take me a while, but I'll get on it. Is the HEAD generally reasonable, or is there a particular commit I should build from?

@greenc-FNAL
Copy link
Collaborator Author

Bug is reproduced on 4f0450b.

@greenc-FNAL
Copy link
Collaborator Author

Bump?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants