Navigation Menu

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

Utilities/StaticAnalyzers: adjust to LLVM/Clang 3.8.0 #13260

Merged
merged 1 commit into from Feb 12, 2016

Conversation

davidlt
Copy link
Contributor

@davidlt davidlt commented Feb 12, 2016

See the following Clang commits:

Error are here: https://cmssdt.cern.ch/SDT/cgi-bin/buildlogs/slc6_amd64_gcc493/CMSSW_8_0_DEVEL_X_2016-02-11-2300/Utilities/StaticAnalyzers

Signed-off-by: David Abdurachmanov David.Abdurachmanov@cern.ch

See the following Clang commits:
- 440c44a96ab56806e40ee260f327fbeaea47b265
- 34d3857a24cef94be9b277803ad4f25574f3489a

Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
@cmsbuild
Copy link
Contributor

A new Pull Request was created by @davidlt for CMSSW_8_0_ROOT64_X.

It involves the following packages:

Utilities/StaticAnalyzers

@cmsbuild, @smuzaffar, @Dr15Jones can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @wddgit this is something you requested to watch as well.
@slava77, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

@davidlt
Copy link
Contributor Author

davidlt commented Feb 12, 2016

@gartung could you double check? In particular arg_begin() usage. I couldn't fully figure out what was happening, thus I looked for usage pattern within Clang and that seems to work.

@gartung
Copy link
Member

gartung commented Feb 12, 2016

Look OK to me.

davidlt added a commit that referenced this pull request Feb 12, 2016
Utilities/StaticAnalyzers: adjust to LLVM/Clang 3.8.0
@davidlt davidlt merged commit d22e94e into cms-sw:CMSSW_8_0_ROOT64_X Feb 12, 2016
@davidlt
Copy link
Contributor Author

davidlt commented Feb 12, 2016

Then I am putting this into DEVEL IBs. Would be nice if you could double check that things work as expected once IB arrives.

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

Successfully merging this pull request may close these issues.

None yet

3 participants