Re: [PATCH] D22075: [OpenMP] Fix incorrect diagnostics in map clause

2016-07-13 Thread Alexey Bataev via cfe-commits
ABataev accepted this revision. ABataev added a comment. This revision is now accepted and ready to land. LG http://reviews.llvm.org/D22075 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co

Re: [PATCH] D21545: CodeGen: Replace ThinLTO backend implementation with a client of LTO/Resolution.

2016-07-13 Thread Peter Collingbourne via cfe-commits
pcc updated this revision to Diff 63908. pcc added a comment. Refresh http://reviews.llvm.org/D21545 Files: lib/CodeGen/BackendUtil.cpp lib/CodeGen/CMakeLists.txt Index: lib/CodeGen/CMakeLists.txt === --- lib/CodeGen/CMakeList

[PATCH] D22334: Fix for Bug 28172 : clang crashes on invalid code (with too few arguments to __builtin_signbit) without any proper diagnostics.

2016-07-13 Thread Mayur Pandey via cfe-commits
mayurpandey created this revision. mayurpandey added reviewers: george.burgess.iv, hfinkel. mayurpandey added a subscriber: cfe-commits. clang was crashing when no argument was provided to __builtin_signbit function. Added a check to handle the error. No regressions on testing. http://reviews.ll

Re: [PATCH] D21983: Add C++ dependencies to xray runtime

2016-07-13 Thread Dean Michael Berris via cfe-commits
dberris updated this revision to Diff 63916. dberris added a comment. Rebase http://reviews.llvm.org/D21983 Files: lib/CodeGen/CodeGenFunction.cpp lib/Driver/Tools.cpp Index: lib/Driver/Tools.cpp === --- lib/Driver/Tools.cpp +

Re: [PATCH] D21983: Add C++ dependencies to xray runtime

2016-07-13 Thread Dean Michael Berris via cfe-commits
dberris updated this revision to Diff 63917. dberris added a comment. - Undo unnecessary formatting change http://reviews.llvm.org/D21983 Files: lib/Driver/Tools.cpp Index: lib/Driver/Tools.cpp === --- lib/Driver/Tools.cpp +++ l

r275368 - Add C++ dependencies to xray runtime

2016-07-13 Thread Dean Michael Berris via cfe-commits
Author: dberris Date: Wed Jul 13 23:58:44 2016 New Revision: 275368 URL: http://llvm.org/viewvc/llvm-project?rev=275368&view=rev Log: Add C++ dependencies to xray runtime Summary: Depends on D21982 which implements the in-memory logging implementation of the XRay runtime. These additional changes

Re: [PATCH] D21983: Add C++ dependencies to xray runtime

2016-07-13 Thread Dean Michael Berris via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL275368: Add C++ dependencies to xray runtime (authored by dberris). Changed prior to commit: http://reviews.llvm.org/D21983?vs=63917&id=63919#toc Repository: rL LLVM http://reviews.llvm.org/D21983

[PATCH] D22338: Use hasFlag instead of hasArg

2016-07-13 Thread Dean Michael Berris via cfe-commits
dberris created this revision. dberris added a reviewer: echristo. dberris added a subscriber: cfe-commits. Herald added a subscriber: mehdi_amini. Fix the build to use hasFlag instead of hasArg for checking some flags. http://reviews.llvm.org/D22338 Files: lib/Driver/Tools.cpp Index: lib/Dri

Re: [PATCH] D22338: Use hasFlag instead of hasArg

2016-07-13 Thread Eric Christopher via cfe-commits
echristo accepted this revision. echristo added a comment. This revision is now accepted and ready to land. LGTM. -eric http://reviews.llvm.org/D22338 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/lis

Re: [PATCH] D22129: [clang-rename] add documentation

2016-07-13 Thread Kirill Bobyrev via cfe-commits
omtcyf0 added a comment. Any other objections? http://reviews.llvm.org/D22129 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r275377 - Use hasFlag instead of hasArg

2016-07-13 Thread Dean Michael Berris via cfe-commits
Author: dberris Date: Thu Jul 14 01:37:46 2016 New Revision: 275377 URL: http://llvm.org/viewvc/llvm-project?rev=275377&view=rev Log: Use hasFlag instead of hasArg Summary: Fix the build to use hasFlag instead of hasArg for checking some flags. Reviewers: echristo Subscribers: mehdi_amini, cfe-

Re: [PATCH] D22338: Use hasFlag instead of hasArg

2016-07-13 Thread Dean Michael Berris via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL275377: Use hasFlag instead of hasArg (authored by dberris). Changed prior to commit: http://reviews.llvm.org/D22338?vs=63922&id=63923#toc Repository: rL LLVM http://reviews.llvm.org/D22338 Files:

Re: [PATCH] D22091: [clang-rename] exit code-related bugfix and code cleanup

2016-07-13 Thread Kirill Bobyrev via cfe-commits
omtcyf0 added a comment. Does it look good now? http://reviews.llvm.org/D22091 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

<    1   2