[PATCH] D39331: Switch -mcount and -finstrument-functions to emit EnterExitInstrumenter attributes

hans updated this revision to Diff 122889. hans edited the summary of this revision. hans added a comment. Renaming the new flag and making it sufficient to turn on instrumentation by itself. https://reviews.llvm.org/D39331 Files: include/clang/Driver/Options.td

[PATCH] D39331: Switch -mcount and -finstrument-functions to emit EnterExitInstrumenter attributes

hans added inline comments. Comment at: include/clang/Driver/Options.td:1032 +def fno_instrument_functions_inline : Flag<["-"], "fno-instrument-functions-inline">, Group, Flags<[CC1Option]>, + HelpText<"When using -finstrument-functions, insert the calls after inlining">;

[PATCH] D39331: Switch -mcount and -finstrument-functions to emit EnterExitInstrumenter attributes

rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land. Thanks, looks good! https://reviews.llvm.org/D39331 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D39505: [OpenMP] Show error if VLAs are not supported

Hahnfeld added inline comments. Comment at: include/clang/Basic/TargetInfo.h:944 + /// \brief Whether target supports variable-length arrays. + bool isVLASupported() const { return VLASupported; } + rjmccall wrote: > Hahnfeld wrote: > > rjmccall wrote: > > >

r318215 - [refactor][selection] canonicalize decl ref callee to the call expr

Author: arphaman Date: Tue Nov 14 15:10:50 2017 New Revision: 318215 URL: http://llvm.org/viewvc/llvm-project?rev=318215=rev Log: [refactor][selection] canonicalize decl ref callee to the call expr We would like to extract the full call when just the callee function is selected Modified:

r318214 - Try to fix the instrument-functions tests

Author: hans Date: Tue Nov 14 15:10:04 2017 New Revision: 318214 URL: http://llvm.org/viewvc/llvm-project?rev=318214=rev Log: Try to fix the instrument-functions tests On e.g. PPC the return value and argument were marked 'signext'. This makes the test expectations a bit more flexible.

[PATCH] D39347: Fix __has_unique_object_representations based on rsmith's input

majnemer added a comment. It'd be good to have tests which have alignment directives on bitfields. https://reviews.llvm.org/D39347 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D39505: [OpenMP] Show error if VLAs are not supported

ABataev added inline comments. Comment at: include/clang/Basic/TargetInfo.h:944 + /// \brief Whether target supports variable-length arrays. + bool isVLASupported() const { return VLASupported; } + Hahnfeld wrote: > rjmccall wrote: > > Hahnfeld wrote: > > >

[PATCH] D39673: Toolchain: Normalize dwarf, sjlj and seh eh

rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land. Looks good, sorry for the delay! Comment at: include/clang/Basic/LangOptions.def:129 LANGOPT(SjLjExceptions, 1, 0, "setjmp-longjump exception handling")

r318205 - [refactor][selection] canonicalize member expr callee to the full

Author: arphaman Date: Tue Nov 14 14:06:55 2017 New Revision: 318205 URL: http://llvm.org/viewvc/llvm-project?rev=318205=rev Log: [refactor][selection] canonicalize member expr callee to the full member call expression We would like to extract the full call when just the callee is selected.

[PATCH] D39446: [PGO] Detect more structural changes with the stable hash

vsk added a comment. Thanks for the review Alex. There hasn't been any more feedback so I'll commit this soon. https://reviews.llvm.org/D39446 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D39879: AMDGPU: Remove -mamdgpu-debugger-abi option

kzhuravl added a comment. Ping. https://reviews.llvm.org/D39879 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D39505: [OpenMP] Show error if VLAs are not supported

rjmccall added inline comments. Comment at: include/clang/Basic/TargetInfo.h:944 + /// \brief Whether target supports variable-length arrays. + bool isVLASupported() const { return VLASupported; } + Hahnfeld wrote: > rjmccall wrote: > > Hahnfeld wrote: > > >

r318194 - [OPENMP] Fix DSA analysis for threadprivates after deserialization.

Author: abataev Date: Tue Nov 14 13:01:01 2017 New Revision: 318194 URL: http://llvm.org/viewvc/llvm-project?rev=318194=rev Log: [OPENMP] Fix DSA analysis for threadprivates after deserialization. If threadprivate vaible is deserialized, it is not marked as threadprivate in DSAStack. Modified:

[libclc] r318204 - native_powr: Switch implementation to native_exp2 and native_log2

Author: jvesely Date: Tue Nov 14 13:55:41 2017 New Revision: 318204 URL: http://llvm.org/viewvc/llvm-project?rev=318204=rev Log: native_powr: Switch implementation to native_exp2 and native_log2 v2: don't use assume check only for x<0, the other conditions are handled transparently v3: don't

[PATCH] D39571: [clangd] DidChangeConfiguration Notification

Nebiroth marked 18 inline comments as done. Nebiroth added inline comments. Comment at: clangd/Protocol.h:285 + + DidChangeConfigurationParams() {} + malaperle wrote: > I don't think you need this constructor? I do inside parse() for

[libcxx] r318208 - Add two new macros: _LIBCPP_NODISCARD_AFTER_CXX17 and _LIBCPP_CONSTEXPR_AFTER_CXX17, along with a way to turn off the NODISCARD one: _LIBCPP_DISABLE_NODISCARD_AFTER_CXX17. No one is

Author: marshall Date: Tue Nov 14 14:26:50 2017 New Revision: 318208 URL: http://llvm.org/viewvc/llvm-project?rev=318208=rev Log: Add two new macros: _LIBCPP_NODISCARD_AFTER_CXX17 and _LIBCPP_CONSTEXPR_AFTER_CXX17, along with a way to turn off the NODISCARD one:

[PATCH] D40045: AMDGPU/GCN: Bring processors in sync with AMDGPUUsage

kzhuravl created this revision. Herald added subscribers: tpr, dstuttard, yaxunl, nhaehnle, wdng. - Remove gfx804 - Remove gfx901 - Remove gfx903 https://reviews.llvm.org/D40045 Files: lib/Basic/Targets/AMDGPU.cpp test/Driver/amdgpu-mcpu.cl Index: test/Driver/amdgpu-mcpu.cl

[PATCH] D39331: Switch -mcount and -finstrument-functions to emit EnterExitInstrumenter attributes

This revision was automatically updated to reflect the committed changes. Closed by commit rL318199: Switch -mcount and -finstrument-functions to emit EnterExitInstrumenter… (authored by hans). Changed prior to commit: https://reviews.llvm.org/D39331?vs=122889=122901#toc Repository: rL LLVM

[PATCH] D40023: [RISCV] Implement ABI lowering

efriedma added reviewers: efriedma, rjmccall. efriedma added a comment. You need more test coverage for the cases where arguments end up on the stack. And some test coverage for varargs calls. Comment at: lib/CodeGen/TargetInfo.cpp:8858 + else +NeededArgGPRs = 1; +

r318199 - Switch -mcount and -finstrument-functions to emit EnterExitInstrumenter attributes

Author: hans Date: Tue Nov 14 13:13:27 2017 New Revision: 318199 URL: http://llvm.org/viewvc/llvm-project?rev=318199=rev Log: Switch -mcount and -finstrument-functions to emit EnterExitInstrumenter attributes This updates -mcount to use the new attribute names (LLVM r318195), and switches over

[PATCH] D39430: [clangd] formatting: don't ignore style

rwols updated this revision to Diff 122904. rwols marked an inline comment as done. rwols added a comment. Removed braces around single statement if-else. https://reviews.llvm.org/D39430 Files: clangd/ClangdLSPServer.cpp clangd/ClangdServer.cpp clangd/ClangdServer.h

[PATCH] D39521: [x86 TargetInfo] Pull CPU handling for the x86 TargetInfo into a .def file.

erichkeane updated this revision to Diff 122934. erichkeane added a comment. Re-ran clang-format, which now has more sane decisions in CGBuiltin.cpp https://reviews.llvm.org/D39521 Files: lib/Basic/Targets/X86.cpp lib/CodeGen/CGBuiltin.cpp Index: lib/CodeGen/CGBuiltin.cpp

[libcxx] r318259 - Named the macro wrong in the test.

Author: marshall Date: Tue Nov 14 19:08:39 2017 New Revision: 318259 URL: http://llvm.org/viewvc/llvm-project?rev=318259=rev Log: Named the macro wrong in the test. Modified: libcxx/trunk/test/libcxx/diagnostics/nodiscard.pass.cpp Modified:

[PATCH] D40062: Update tests for llvm.invariant.group.barrier becoming mangled

arsenm accepted this revision. arsenm added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D40062 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D40054: Simplify CpuIs code to use include from LLVM

erichkeane added inline comments. Comment at: lib/CodeGen/CGBuiltin.cpp:4630 case NEON::BI__builtin_neon_vsha1mq_u32: - case ARM::BI_MoveToCoprocessor: - case ARM::BI_MoveToCoprocessor2: + case clang::ARM::BI_MoveToCoprocessor: + case clang::ARM::BI_MoveToCoprocessor2:

r318229 - [PGO] Detect more structural changes with the stable hash

Author: vedantk Date: Tue Nov 14 15:56:53 2017 New Revision: 318229 URL: http://llvm.org/viewvc/llvm-project?rev=318229=rev Log: [PGO] Detect more structural changes with the stable hash Lifting from Bob Wilson's notes: The hash value that we compute and store in PGO profile data to detect

[PATCH] D39446: [PGO] Detect more structural changes with the stable hash

This revision was automatically updated to reflect the committed changes. Closed by commit rL318229: [PGO] Detect more structural changes with the stable hash (authored by vedantk). Changed prior to commit: https://reviews.llvm.org/D39446?vs=122368=122939#toc Repository: rL LLVM

r318234 - Simplify CpuIs code to use include from LLVM

Author: erichkeane Date: Tue Nov 14 16:11:24 2017 New Revision: 318234 URL: http://llvm.org/viewvc/llvm-project?rev=318234=rev Log: Simplify CpuIs code to use include from LLVM LLVM exposes a file in the backend (X86TargetParser.def) that contains information about the correct list of CpuIs

[PATCH] D40054: Simplify CpuIs code to use include from LLVM

This revision was automatically updated to reflect the committed changes. Closed by commit rL318234: Simplify CpuIs code to use include from LLVM (authored by erichkeane). Changed prior to commit: https://reviews.llvm.org/D40054?vs=122936=122942#toc Repository: rL LLVM

[PATCH] D38818: Template Instantiation Observer + a few other templight-related changes

rsmith closed this revision. rsmith added a comment. Let's keep the complete review thread together on https://reviews.llvm.org/D5767. https://reviews.llvm.org/D38818 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D5767: Template Instantiation Observer + a few other templight-related changes

rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D5767 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D39913: [ObjC] warn about availability attributes missing from a method's declaration when they're specified for a method's definition

arphaman updated this revision to Diff 122949. arphaman added a comment. Remove C++ support for now (it will be in a followup patch along with non-member function support). Repository: rL LLVM https://reviews.llvm.org/D39913 Files: include/clang/Basic/DiagnosticSemaKinds.td

r318245 - [modules] Fix crash in complex class merging scenario.

Author: rsmith Date: Tue Nov 14 17:33:46 2017 New Revision: 318245 URL: http://llvm.org/viewvc/llvm-project?rev=318245=rev Log: [modules] Fix crash in complex class merging scenario. When we merge together class definitions, we can end up with the canonical declaration of a field not being the

[libcxx] r318244 - Added tests for xxx.size() and xxx.empty() for all the sequence containers

Author: marshall Date: Tue Nov 14 17:33:33 2017 New Revision: 318244 URL: http://llvm.org/viewvc/llvm-project?rev=318244=rev Log: Added tests for xxx.size() and xxx.empty() for all the sequence containers Added: libcxx/trunk/test/std/containers/sequences/deque/deque.capacity/empty.pass.cpp

[PATCH] D39913: [ObjC] warn about availability attributes missing from a method's declaration when they're specified for a method's definition

arphaman added a comment. In https://reviews.llvm.org/D39913#924131, @ahatanak wrote: > Is it not necessary to print a diagnostic when a non-member function > declaration is missing an availability attribute? > > void foo1(); > > __attribute__((availability(macos, introduced=10.1))) >

[libcxx] r318247 - Update Appveyor LLVM install link to be current and non-broken

Author: ericwf Date: Tue Nov 14 17:37:11 2017 New Revision: 318247 URL: http://llvm.org/viewvc/llvm-project?rev=318247=rev Log: Update Appveyor LLVM install link to be current and non-broken Modified: libcxx/trunk/appveyor-reqs-install.cmd Modified: libcxx/trunk/appveyor-reqs-install.cmd

[PATCH] D39347: Fix __has_unique_object_representations based on rsmith's input

erichkeane updated this revision to Diff 122897. erichkeane added a comment. Added a test for aligned bitfield, as @majnemer requested. Is this sufficient? https://reviews.llvm.org/D39347 Files: include/clang/AST/ASTContext.h include/clang/AST/Type.h lib/AST/ASTContext.cpp

r318216 - [AST] Fix some Clang-tidy modernize and Include What You Use warnings; other minor fixes (NFC).

Author: eugenezelenko Date: Tue Nov 14 15:13:32 2017 New Revision: 318216 URL: http://llvm.org/viewvc/llvm-project?rev=318216=rev Log: [AST] Fix some Clang-tidy modernize and Include What You Use warnings; other minor fixes (NFC). Modified: cfe/trunk/include/clang/AST/DeclCXX.h

[PATCH] D40054: Simplify CpuIs code to use include from LLVM

erichkeane updated this revision to Diff 122936. erichkeane added a comment. Re-ran format on a few lines to make them more sane. https://reviews.llvm.org/D40054 Files: lib/Basic/Targets/X86.cpp lib/CodeGen/CGBuiltin.cpp Index: lib/CodeGen/CGBuiltin.cpp

[PATCH] D39521: [x86 TargetInfo] Pull CPU handling for the x86 TargetInfo into a .def file.

erichkeane abandoned this revision. erichkeane added a comment. First, updated this incorrectly. Second, going to do this work over a few patches to take advantage of work that Craig did in the backend. https://reviews.llvm.org/D39521 ___

[PATCH] D40058: add check to avoid throwing objc exception according to Google Objective-C guide

Wizard updated this revision to Diff 122953. Wizard added a comment. new line https://reviews.llvm.org/D40058 Files: clang-tidy/google/AvoidThrowingObjcExceptionCheck.cpp clang-tidy/google/AvoidThrowingObjcExceptionCheck.h clang-tidy/google/CMakeLists.txt

[PATCH] D40060: [clangd] Fuzzy match scorer

sammccall created this revision. Herald added a subscriber: mgorny. This will be used for rescoring code completion results based on partial identifiers. Short-term use: - we want to limit the number of code completion results returned to improve performance of global completion. The scorer

[PATCH] D39579: BuiltinOperatorOverloadBuilder: Don't consider types that are unavailable on the target (PR35174)

rsmith accepted this revision. rsmith added inline comments. This revision is now accepted and ready to land. Comment at: lib/Sema/SemaOverload.cpp:7618 - // Define some constants used to index and iterate over the arithemetic types - // provided via the getArithmeticType()

[PATCH] D40060: [clangd] Fuzzy match scorer

sammccall updated this revision to Diff 122956. sammccall added a comment. clang-format https://reviews.llvm.org/D40060 Files: clangd/CMakeLists.txt clangd/FuzzyMatch.cpp clangd/FuzzyMatch.h unittests/clangd/CMakeLists.txt unittests/clangd/FuzzyMatchTests.cpp Index:

[libcxx] r318256 - More missing tests - array<>::size() and array<>::max_size()

Author: marshall Date: Tue Nov 14 18:31:14 2017 New Revision: 318256 URL: http://llvm.org/viewvc/llvm-project?rev=318256=rev Log: More missing tests - array<>::size() and array<>::max_size() Added: libcxx/trunk/test/std/containers/sequences/array/empty.pass.cpp

r318258 - PR35214: don't crash if we see an array of unknown bound added to an empty but invalid designator.

Author: rsmith Date: Tue Nov 14 19:03:56 2017 New Revision: 318258 URL: http://llvm.org/viewvc/llvm-project?rev=318258=rev Log: PR35214: don't crash if we see an array of unknown bound added to an empty but invalid designator. Modified: cfe/trunk/lib/AST/ExprConstant.cpp

r318221 - [AST] Fix some Clang-tidy modernize and Include What You Use warnings; other minor fixes (NFC).

Author: eugenezelenko Date: Tue Nov 14 15:35:42 2017 New Revision: 318221 URL: http://llvm.org/viewvc/llvm-project?rev=318221=rev Log: [AST] Fix some Clang-tidy modernize and Include What You Use warnings; other minor fixes (NFC). Modified: cfe/trunk/include/clang/AST/Stmt.h

[PATCH] D40054: Simplify CpuIs code to use include from LLVM

erichkeane created this revision. LLVM exposes a file in the backend (X86TargetParser.def) that contains information about the correct list of CpuIs values. This patch removes 2 of the copied and pasted versions of this list from clang and instead includes the data from the .def file.

[PATCH] D40062: Update tests for llvm.invariant.group.barrier becoming mangled

yaxunl created this revision. Herald added subscribers: Prazek, wdng. https://reviews.llvm.org/D40062 Files: test/CodeGenCXX/invariant.group-for-vptrs.cpp test/CodeGenCXX/strict-vtable-pointers.cpp Index: test/CodeGenCXX/strict-vtable-pointers.cpp

[PATCH] D39430: [clangd] formatting: don't ignore style

rwols marked 2 inline comments as done. rwols added inline comments. Comment at: clangd/ClangdServer.h:289 + llvm::Expected + formatRange(llvm::StringRef Code, PathRef File, Range Rng); + ilya-biryukov wrote: > Why do we accept `Code` as a

[libcxx] r318276 - Tolerate [[nodiscard]] annotations in the STL. Reviewed as https://reviews.llvm.org/D39033

Author: bion Date: Tue Nov 14 23:40:37 2017 New Revision: 318276 URL: http://llvm.org/viewvc/llvm-project?rev=318276=rev Log: Tolerate [[nodiscard]] annotations in the STL. Reviewed as https://reviews.llvm.org/D39033 Modified:

[libcxx] r318277 - Tolerate even more [[nodiscard]] in the STL. Reviewed as https://reviews.llvm.org/D39080

Author: bion Date: Tue Nov 14 23:45:07 2017 New Revision: 318277 URL: http://llvm.org/viewvc/llvm-project?rev=318277=rev Log: Tolerate even more [[nodiscard]] in the STL. Reviewed as https://reviews.llvm.org/D39080 Modified:

[libclc] r318264 - math: Implement maxmag

Author: jvesely Date: Tue Nov 14 20:10:37 2017 New Revision: 318264 URL: http://llvm.org/viewvc/llvm-project?rev=318264=rev Log: math: Implement maxmag Reviewer: Aaron Watry Signed-off-by: Jan Vesely Added: libclc/trunk/generic/include/clc/math/maxmag.h

[libclc] r318265 - math: Implement minmag

Author: jvesely Date: Tue Nov 14 20:10:39 2017 New Revision: 318265 URL: http://llvm.org/viewvc/llvm-project?rev=318265=rev Log: math: Implement minmag Reviewer: Aaron Watry Signed-off-by: Jan Vesely Added: libclc/trunk/generic/include/clc/math/minmag.h

r318274 - ASTMatchers.h: Avoid warnings due to "@throw". [-Wdocumentation]

Author: chapuni Date: Tue Nov 14 22:53:45 2017 New Revision: 318274 URL: http://llvm.org/viewvc/llvm-project?rev=318274=rev Log: ASTMatchers.h: Avoid warnings due to "@throw". [-Wdocumentation] Modified: cfe/trunk/include/clang/ASTMatchers/ASTMatchers.h Modified:

[libcxx] r318268 - Still more missing tests - this time for the unordered containers

Author: marshall Date: Tue Nov 14 21:25:36 2017 New Revision: 318268 URL: http://llvm.org/viewvc/llvm-project?rev=318268=rev Log: Still more missing tests - this time for the unordered containers Added: libcxx/trunk/test/std/containers/unord/unord.map/empty.pass.cpp

[libcxx] r318269 - First part of P0600 - '[[nodiscard] in the standard library'. Mark the 'empty()' methods of all the containers as nodiscard. If you're calling empty() w/o looking at the result, you

Author: marshall Date: Tue Nov 14 21:51:26 2017 New Revision: 318269 URL: http://llvm.org/viewvc/llvm-project?rev=318269=rev Log: First part of P0600 - '[[nodiscard] in the standard library'. Mark the 'empty()' methods of all the containers as nodiscard. If you're calling empty() w/o looking at

Clang-format: add finer-grained options for putting all arguments on one line.

Attached is a patch that adds two new options, AllowAllArgumentsOnNextLine and AllowAllConstructorInitializersOnNextLine. These mirror the existing AllowAllParametersOfDeclarationOnNextLine and allow me to support an internal style guide where I work. I think this would be generally useful, some

[libcxx] r318144 - Rename identifiers named `__output`

Author: arichardson Date: Tue Nov 14 03:14:25 2017 New Revision: 318144 URL: http://llvm.org/viewvc/llvm-project?rev=318144=rev Log: Rename identifiers named `__output` Summary: In the CHERI clang compiler __output and __input are keywords and therefore we can't compile libc++ with our compiler.

[PATCH] D39049: [analyzer] Fix wrong calculation of offset in ArrayBoundsV2

xazax.hun added a comment. In https://reviews.llvm.org/D39049#910482, @NoQ wrote: > // TODO: once the constraint manager is smart enough to handle non > simplified > // symbolic expressions remove this function. Note that this can not be > used in > // the constraint manager as is, since

[PATCH] D36836: [clang-tidy] Implement readability-function-cognitive-complexity check

lebedev.ri added a comment. Ping. In https://reviews.llvm.org/D36836#889375, @aaron.ballman wrote: > Adding @dberlin for licensing discussion questions. @dberlin ping? I'm wondering if you had the chance to look at this? :) Repository: rL LLVM https://reviews.llvm.org/D36836

r318150 - Make DiagnosticIDs::getAllDiagnostics static. NFC.

Author: xazax Date: Tue Nov 14 04:14:49 2017 New Revision: 318150 URL: http://llvm.org/viewvc/llvm-project?rev=318150=rev Log: Make DiagnosticIDs::getAllDiagnostics static. NFC. Patch by: Andras Leitereg! Differential Revision: https://reviews.llvm.org/D39372 Modified:

[PATCH] D39372: Make DiagnosticIDs::getAllDiagnostics static.

This revision was automatically updated to reflect the committed changes. Closed by commit rL318150: Make DiagnosticIDs::getAllDiagnostics static. NFC. (authored by xazax). Changed prior to commit: https://reviews.llvm.org/D39372?vs=121104=122822#toc Repository: rL LLVM

r318140 - [NewPassManager] Pass the -fdebug-pass-manager flag setting into the Analysis managers to match what we do in opt

Author: ctopper Date: Tue Nov 14 00:48:28 2017 New Revision: 318140 URL: http://llvm.org/viewvc/llvm-project?rev=318140=rev Log: [NewPassManager] Pass the -fdebug-pass-manager flag setting into the Analysis managers to match what we do in opt Summary: Currently the -fdebug-pass-manager flag for

[PATCH] D40007: [NewPassManager] Pass the -fdebug-pass-manager flag setting into the Analysis managers to match what we do in opt

This revision was automatically updated to reflect the committed changes. Closed by commit rL318140: [NewPassManager] Pass the -fdebug-pass-manager flag setting into the Analysis… (authored by ctopper). Changed prior to commit: https://reviews.llvm.org/D40007?vs=122788=122792#toc Repository:

[PATCH] D39829: add new check for property declaration

hokein added inline comments. Comment at: clang-tidy/objc/PropertyDeclarationCheck.cpp:89 + assert(MatchedDecl->getName().size() > 0); + // Skip the check of lowerCamelCase if the name has prefix of special acronyms + if (startsWithSpecialAcronyms(MatchedDecl->getName(),

[PATCH] D40007: [NewPassManager] Pass the -fdebug-pass-manager flag setting into the Analysis managers to match what we do in opt

chandlerc accepted this revision. chandlerc added a comment. This revision is now accepted and ready to land. LGTM, nice catch! Maybe update one of the new PM tests to check that this debug printing is available? =D https://reviews.llvm.org/D40007

[PATCH] D40007: [NewPassManager] Pass the -fdebug-pass-manager flag setting into the Analysis managers to match what we do in opt

craig.topper created this revision. Currently the -fdebug-pass-manager flag for clang doesn't enable the debug logging in the analysis managers. This is different than what the switch does when passed to opt. https://reviews.llvm.org/D40007 Files: lib/CodeGen/BackendUtil.cpp Index:

r318141 - Refactor ContinuationIndenter's breakProtrudingToken logic.

Author: klimek Date: Tue Nov 14 01:19:53 2017 New Revision: 318141 URL: http://llvm.org/viewvc/llvm-project?rev=318141=rev Log: Refactor ContinuationIndenter's breakProtrudingToken logic. Create more orthogonal pieces. The restructuring made it easy to try out several alternatives to D33589, and

[PATCH] D39900: Refactor ContinuationIndenter's breakProtrudingToken logic into slightly moreorthogonal pieces.

This revision was automatically updated to reflect the committed changes. Closed by commit rL318141: Refactor ContinuationIndenter's breakProtrudingToken logic. (authored by klimek). Repository: rL LLVM https://reviews.llvm.org/D39900 Files: cfe/trunk/lib/Format/BreakableToken.cpp

[PATCH] D33589: clang-format: consider not splitting tokens in optimization

klimek added a comment. In https://reviews.llvm.org/D33589#920160, @Typz wrote: > ping ? I'm working on understanding this better :) I've refactored the code a bit so I could fully understand the problem, which I now do (sorry for this taking a while, but it took me multiple hours to work

[PATCH] D39537: Rename identifiers named `__output`

This revision was automatically updated to reflect the committed changes. Closed by commit rL318144: Rename identifiers named `__output` (authored by arichardson). Repository: rL LLVM https://reviews.llvm.org/D39537 Files: libcxx/trunk/include/algorithm

r318142 - [libclang] Allow crash recovery with LIBCLANG_NOTHREADS

Author: erikjv Date: Tue Nov 14 01:34:39 2017 New Revision: 318142 URL: http://llvm.org/viewvc/llvm-project?rev=318142=rev Log: [libclang] Allow crash recovery with LIBCLANG_NOTHREADS Enabled crash recovery for some libclang operations on a calling thread even when LIBCLANG_NOTHREAD is

[PATCH] D39129: [OpenCL] Fix generation of constant address space sampler in function scope

bader added a comment. In https://reviews.llvm.org/D39129#923235, @Anastasia wrote: > In https://reviews.llvm.org/D39129#902848, @bader wrote: > > > @Anastasia, during the discussion of similar fix > > (https://reviews.llvm.org/D34342). > > > > I found another bug in the CodeGen library. Do you

[PATCH] D39900: Refactor ContinuationIndenter's breakProtrudingToken logic into slightly moreorthogonal pieces.

klimek added a comment. In https://reviews.llvm.org/D39900#921885, @krasimir wrote: > Maybe we should further refactor `getRawStringStyle` into > `llvm::Optional> getRawStringStyleAndDelimiter` > and that would nicely take care of the duplicated effort? I thought

[PATCH] D40016: Use ImplicitConversionSequence::setAsIdentityConversion(QualType). NFC

arichardson created this revision. Slightly reduces the amount of duplicated code. https://reviews.llvm.org/D40016 Files: lib/Sema/SemaOverload.cpp Index: lib/Sema/SemaOverload.cpp === --- lib/Sema/SemaOverload.cpp +++

[PATCH] D37554: [libclang] Allow crash recovery with LIBCLANG_NOTHREADS

erikjv closed this revision. erikjv added a comment. Committed as r318142. https://reviews.llvm.org/D37554 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D38680: [libunwind] Fix handling of DW_CFA_GNU_args_size

mstorsjo added a comment. Ping @joerg https://reviews.llvm.org/D38680 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r318147 - [ASTImporter] TypeAliasTemplate and PackExpansion importing capability

Author: xazax Date: Tue Nov 14 03:30:38 2017 New Revision: 318147 URL: http://llvm.org/viewvc/llvm-project?rev=318147=rev Log: [ASTImporter] TypeAliasTemplate and PackExpansion importing capability Patch by: Zoltan Gera! Differential Revision: https://reviews.llvm.org/D39247 Modified:

[PATCH] D40024: Fix skipping of flags in getClangStripDependencyFileAdjuster

kastiglione created this revision. The ArgumentsAdjuster returned from `getClangStripDependencyFileAdjuster` will skip dependency flags, and also their associated values for those flags that take an argument. This change corrects the handling of the `-MD` and `-MMD` flags, which do not take an

[PATCH] D39963: [RISCV][RFC] Add initial RISC-V target and driver support

asb added inline comments. Comment at: lib/Driver/ToolChains/RISCV.cpp:86 +CmdArgs.push_back("-lc"); +CmdArgs.push_back("-lgloss"); +CmdArgs.push_back("--end-group"); apazos wrote: > mgrang wrote: > > How about if our sysroot is linux (as opposed to

[PATCH] D39948: Make isDefinition matcher support ObjCMethodDecl

kastiglione updated this revision to Diff 122831. kastiglione added a comment. Regenerate AST Matcher docs https://reviews.llvm.org/D39948 Files: docs/LibASTMatchersReference.html include/clang/ASTMatchers/ASTMatchers.h unittests/ASTMatchers/ASTMatchersNarrowingTest.cpp Index:

[PATCH] D39948: Make isDefinition matcher support ObjCMethodDecl

This revision was automatically updated to reflect the committed changes. Closed by commit rL318152: Make isDefinition matcher support ObjCMethodDecl (authored by kastiglione). Repository: rL LLVM https://reviews.llvm.org/D39948 Files: cfe/trunk/docs/LibASTMatchersReference.html

[PATCH] D39505: [OpenMP] Show error if VLAs are not supported

Hahnfeld added inline comments. Comment at: include/clang/Basic/TargetInfo.h:944 + /// \brief Whether target supports variable-length arrays. + bool isVLASupported() const { return VLASupported; } + rjmccall wrote: > Hahnfeld wrote: > > rjmccall wrote: > > >

[PATCH] D39842: Allow to store precompiled preambles in memory.

ilya-biryukov added inline comments. Comment at: lib/Frontend/ASTUnit.cpp:1028 +IntrusiveRefCntPtr OldVFS = VFS; +Preamble->AddImplicitPreamble(*CCInvocation, /*ref*/ VFS, + OverrideMainBuffer.get()); klimek wrote: >

[PATCH] D39843: [clangd] Use in-memory preambles in clangd.

ilya-biryukov updated this revision to Diff 122845. ilya-biryukov added a comment. Made in-memory preambles optional (on-disk by default). https://reviews.llvm.org/D39843 Files: clangd/ClangdLSPServer.cpp clangd/ClangdLSPServer.h clangd/ClangdServer.cpp clangd/ClangdServer.h

[PATCH] D39843: [clangd] Use in-memory preambles in clangd.

ilya-biryukov added a comment. In https://reviews.llvm.org/D39843#920568, @sammccall wrote: > As a very first step, can we make this configurable but off-by-default? That > will address the pressing diskless-servers need. I think on-disk by default makes sense now until we do the measurements

[PATCH] D33589: clang-format: consider not splitting tokens in optimization

klimek added a comment. One interesting trade-off I'm running into: My gut feeling is that we really want to make local decisions about whether we want to break/reflow - this makes the code significantly simpler (IMO), and handles all tests in this patch correctly, but is fundamentally limiting

[PATCH] D39430: [clangd] formatting: don't ignore style

ilya-biryukov added inline comments. Comment at: clangd/ClangdLSPServer.cpp:153 + if (ReplacementsOrError) { +C.reply(json::ary{replacementsToEdits(Code, ReplacementsOrError.get())}); + } else { NIT: remove braces from single-statement branches

[PATCH] D39852: [clangd] Support returning a limited number of completion results.

ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D39852 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D38708: [AST] Flag the typo-corrected nodes for better tooling

ilya-biryukov added a comment. Sorry, not familiar enough with the AST bits yet to LGTM this. Looks good from a higher-level perspective, though. It's a bit of a shame that's it is still super-easy to write code that does not account for typo-corrected nodes. A more explicit approach, similar

[PATCH] D39948: Make isDefinition matcher support ObjCMethodDecl

kastiglione updated this revision to Diff 122830. kastiglione added a comment. Update isDefinition comment docs to reflect addition of ObjCMethodDecl https://reviews.llvm.org/D39948 Files: docs/LibASTMatchersReference.html include/clang/ASTMatchers/ASTMatchers.h

[PATCH] D39842: Allow to store precompiled preambles in memory.

ilya-biryukov updated this revision to Diff 122838. ilya-biryukov marked 2 inline comments as done. ilya-biryukov added a comment. - Fixed comments. - Removed /*ref*/ annotations. - Removed unused "Storage" variable. - Extract a helper function that properly sets up VFS to access the PCHStorage.

[PATCH] D40024: Fix skipping of flags in getClangStripDependencyFileAdjuster

kastiglione updated this revision to Diff 122829. kastiglione added a comment. Add a continue; https://reviews.llvm.org/D40024 Files: lib/Tooling/ArgumentsAdjusters.cpp Index: lib/Tooling/ArgumentsAdjusters.cpp === ---

r318154 - [tooling] Make compile_flags.txt negative test more hermetic

Author: sammccall Date: Tue Nov 14 07:22:34 2017 New Revision: 318154 URL: http://llvm.org/viewvc/llvm-project?rev=318154=rev Log: [tooling] Make compile_flags.txt negative test more hermetic Modified: cfe/trunk/test/Tooling/fixed-database.cpp Modified:

[PATCH] D40023: [RISCV] Implement ABI lowering

asb created this revision. Herald added subscribers: jordy.potman.lists, rbar, arichardson. RISCVABIInfo is implemented in terms of XLen, supporting both RV32 and RV64. Unfortunately we need to count argument registers in the frontend in order to determine when to emit signext and zeroext

[PATCH] D37187: [Analyzer] Fix Bug 25609 - Assertion UNREACHABLE: 'Unexpected ProgramPoint' with widen-loops=true

xazax.hun accepted this revision. xazax.hun added a comment. This revision is now accepted and ready to land. LGTM! But wait for @dcoughlin, @zaks.anna , or @NoQ before commit. https://reviews.llvm.org/D37187 ___ cfe-commits mailing list

r318152 - Make isDefinition matcher support ObjCMethodDecl

Author: kastiglione Date: Tue Nov 14 06:17:26 2017 New Revision: 318152 URL: http://llvm.org/viewvc/llvm-project?rev=318152=rev Log: Make isDefinition matcher support ObjCMethodDecl Summary: Allow the `isDefinition()` matcher to apply to `ObjCMethodDecl` nodes, in addition to those it already

[PATCH] D39331: Switch -mcount and -finstrument-functions to emit EnterExitInstrumenter attributes

rnk added inline comments. Comment at: include/clang/Driver/Options.td:1032 +def fno_instrument_functions_inline : Flag<["-"], "fno-instrument-functions-inline">, Group, Flags<[CC1Option]>, + HelpText<"When using -finstrument-functions, insert the calls after inlining">;

  1   2   >