-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Cxx modules remove root maps #2542
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
Cxx modules remove root maps #2542
Conversation
|
Starting build on |
c17bffc to
67cb561
Compare
|
@phsft-bot build just on slc6/gcc62 with flags -Druntime_cxxmodules=On -Dctest_test_exclude_none=on |
|
Starting build on |
|
Build failed on slc6/gcc62. |
c2a633e to
6179630
Compare
|
@phsft-bot build just on slc6/gcc62 with flags -Druntime_cxxmodules=On -Dctest_test_exclude_none=on |
|
Starting build on |
|
@phsft-bot build just on slc6/gcc62 with flags -Druntime_cxxmodules=On -Droofit=Off -Dctest_test_exclude_none=on |
|
Starting build on |
|
@phsft-bot build just on slc6/gcc62 with flags -Druntime_cxxmodules=On -Droofit=Off -Dctest_test_exclude_none=on |
|
Starting build on |
|
Build failed on slc6/gcc62. Warnings:
|
6179630 to
834cd67
Compare
|
Starting build on |
|
Build failed on mac1014/cxx17. |
|
Build failed on ROOT-centos7/noimt. Errors:
|
|
@pcanal, could you give us a hand here? This PR removes the generation of rootmap files when we have a module. It fails because of similar to this error: Here we are destroying a TGenericClassInfo object which has never been loaded. I would guess that without having rootmap files around we load less classes (i.e. only the ones we actually need). Could you give an example when this happens or even better tell how to fix it. |
|
@phsft-bot build just on ROOT-ubuntu16/rtcxxmod with flags -Dctest_test_exclude_none=on |
|
Starting build on |
|
To reproduce: |
|
@phsft-bot build just on ROOT-ubuntu16/rtcxxmod with flags -Dctest_test_exclude_none=on -DCMAKE_BUILD_TYPE=Debug -DLLVM_BUILD_TYPE=Debug |
|
Starting build on |
No description provided.