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
Workaround ICC bug. #7583
Workaround ICC bug. #7583
Conversation
CMSSW compiled with ICC reports a bunch of link errors of the kind: ``` tmp/slc6_amd64_gcc491/src/HLTrigger/Egamma/src/HLTriggerEgamma/HLTDisplacedEgammaFilter.o:(.data._ZTVN3edm6global8EDFilterIIEEE9HLTFilter__24HLTDisplacedEgammaFilter[_ZTVN3edm6global8EDFilterIIEEE9HLTFilter__24HLTDisplacedEgammaFilter]+0x18): undefined reference to `edm::global::EDFilter<>::~EDFilter()' tmp/slc6_amd64_gcc491/src/HLTrigger/Egamma/src/HLTriggerEgamma/HLTDisplacedEgammaFilter.o:(.data._ZTVN3edm6global8EDFilterIIEEE9HLTFilter__24HLTDisplacedEgammaFilter[_ZTVN3edm6global8EDFilterIIEEE9HLTFilter__24HLTDisplacedEgammaFilter]+0x20): undefined reference to `edm::global::EDFilter<>::~EDFilter()' ``` and similar for other `edm::global::EDFilter<>` derived classes. Notice `edm::global::EDFilterBase`, one of the two parent classes of `edm::global::EDFilter<>`, does have a virtual destructor, so I do believe this is actually a wrong behavior in ICC. Swapping the order of parent classes in `edm::global::EDFilter` and adding an explicit virtual destructor there makes the problem disappear. This is clearly not an acceptable solution, but so far I could not find anything else. This PR is just to keep track of the issue.
A new Pull Request was created by @ktf (Giulio Eulisse) for CMSSW_7_4_X. Workaround ICC bug. It involves the following packages: FWCore/Framework @cmsbuild, @Dr15Jones, @ktf, @nclopezo can you please review it and eventually sign? Thanks. |
@cmsbuild please test |
The tests are being triggered in jenkins. |
It had the Couldn't find remote ref refs/pull/7583/headissue. I restarted the tests using the branch name. |
Can you please fail the test when that happens? |
+1
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @ktf, @smuzaffar |
+1 |
CMSSW compiled with ICC reports a bunch of link errors of the kind:
and similar for other
edm::global::EDFilter<>
derived classes.Notice
edm::global::EDFilterBase
, one of the two parent classes ofedm::global::EDFilter<>
, does have a virtual destructor, so I do believe this is actually a wrong behavior in ICC.Swapping the order of parent classes in
edm::global::EDFilter
and adding an explicit virtual destructor there makes the problem disappear.This is clearly not an acceptable solution, but so far I could not find anything else. This PR is just to keep track of the issue.