r296999 - [XRay] [clang] Allow logging the first argument of a function call.

2017-03-05 Thread Dean Michael Berris via cfe-commits
Author: dberris Date: Mon Mar 6 01:08:21 2017 New Revision: 296999 URL: http://llvm.org/viewvc/llvm-project?rev=296999&view=rev Log: [XRay] [clang] Allow logging the first argument of a function call. Summary: Functions with the "xray_log_args" attribute will tell LLVM to emit a special XRay sle

[PATCH] D29704: [XRay] [clang] Allow logging the first argument of a function call.

2017-03-05 Thread Dean Michael Berris via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL296999: [XRay] [clang] Allow logging the first argument of a function call. (authored by dberris). Changed prior to commit: https://reviews.llvm.org/D29704?vs=90431&id=90651#toc Repository: rL LLVM

[PATCH] D30174: [Sema][ObjC] Warn about 'performSelector' calls with selectors that return record types

2017-03-05 Thread Akira Hatanaka via Phabricator via cfe-commits
ahatanak accepted this revision. ahatanak added a comment. This revision is now accepted and ready to land. Looks good, thanks! Repository: rL LLVM https://reviews.llvm.org/D30174 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lis

r296996 - [ubsan] Extend the nonnull arg check to ObjC

2017-03-05 Thread Vedant Kumar via cfe-commits
Author: vedantk Date: Sun Mar 5 23:28:22 2017 New Revision: 296996 URL: http://llvm.org/viewvc/llvm-project?rev=296996&view=rev Log: [ubsan] Extend the nonnull arg check to ObjC UBSan's nonnull argument check applies when a parameter has the "nonnull" attribute. The check currently works for Fun

[PATCH] D30599: [ubsan] Extend the nonnull argument check to ObjC

2017-03-05 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL296996: [ubsan] Extend the nonnull arg check to ObjC (authored by vedantk). Changed prior to commit: https://reviews.llvm.org/D30599?vs=90564&id=90647#toc Repository: rL LLVM https://reviews.llvm.or

[PATCH] D30599: [ubsan] Extend the nonnull argument check to ObjC

2017-03-05 Thread Vedant Kumar via Phabricator via cfe-commits
vsk marked an inline comment as done. vsk added a comment. Thanks for the reviews! Comment at: lib/CodeGen/CodeGenFunction.h:308 + /// \brief An abstract representation of regular/ObjC call/message targets. + class AbstractCallee { aprantl wrote: > The \bri

[libunwind] r296991 - Drop the dependency on dl_unwind_find_exidx().

2017-03-05 Thread Ed Schouten via cfe-commits
Author: ed Date: Sun Mar 5 13:11:24 2017 New Revision: 296991 URL: http://llvm.org/viewvc/llvm-project?rev=296991&view=rev Log: Drop the dependency on dl_unwind_find_exidx(). While porting libunwind over to CloudABI for ARMv6, I observed that this source file doesn't build, as it depends on dl_u

[PATCH] D30610: [clang-tidy] Added options to cppcoreguidelines-special-member-functions check

2017-03-05 Thread Florian Gross via Phabricator via cfe-commits
fgross updated this revision to Diff 90607. fgross marked an inline comment as done. fgross added a comment. reformatted https://reviews.llvm.org/D30610 Files: clang-tidy/cppcoreguidelines/SpecialMemberFunctionsCheck.cpp clang-tidy/cppcoreguidelines/SpecialMemberFunctionsCheck.h docs/clan

[PATCH] D30607: Replace re module by regex module in run-clang-tidy script

2017-03-05 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh added a comment. Actually, I'm not sure why we need groups in that regex. We can instead try replacing `re.compile('(' + ')|('.join(args.files) + ')')` with `re.compile('|'.join(args.files))`. https://reviews.llvm.org/D30607 ___ cfe-commits

[PATCH] D30607: Replace re module by regex module in run-clang-tidy script

2017-03-05 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh added a comment. https://docs.python.org/2/howto/regex.html#introduction says "The regex module was removed completely in Python 2.5.". Why would we want to switch to it? https://reviews.llvm.org/D30607 ___ cfe-commits mailing list cfe-commi