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
Fix clang build problem in AdaptiveVertexFitter #7195
Fix clang build problem in AdaptiveVertexFitter #7195
Conversation
clang does not permit a variable sized array to be passed to a lambda. So instead we pass a pointer to the first element of the array. An optimizing compiler should make the two implementations into the same machine code.
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_7_4_X. Fix clang build problem in AdaptiveVertexFitter It involves the following packages: RecoVertex/AdaptiveVertexFit @cmsbuild, @cvuosalo, @nclopezo, @slava77 can you please review it and eventually sign? Thanks. |
The package compiles fine under pre-3.6 Clang in DEVEL IB. A number of failing packages in CLANG IB are fine with new Clang IIRC. |
Great, but I'm afraid we need the clang build to be clean now so that the static analyzer report is complete. |
-1 runTheMatrix-results/1306.0_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15+MINIAODMCUP15/step3_SingleMuPt1_UP15+SingleMuPt1_UP15+DIGIUP15+RECOUP15+HARVESTUP15+MINIAODMCUP15.log 25202.0 step3 runTheMatrix-results/25202.0_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15+MINIAODMCUP15/step3_TTbar_13+TTbar_13+DIGIUP15_PU25+RECOUP15_PU25+HARVESTUP15+MINIAODMCUP15.log you can see the results of the tests here: |
The test failures are unrelated to this pull request. |
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 will be automatically merged. |
…rtexFitter Fix clang build problem in AdaptiveVertexFitter
clang does not permit a variable sized array to be passed to a lambda.
So instead we pass a pointer to the first element of the array. An
optimizing compiler should make the two implementations into the same
machine code.