[PATCH] D61697: [lit] Disable test on darwin when building shared libs.

2019-06-07 Thread Sam McCall via Phabricator via cfe-commits
sammccall resigned from this revision. sammccall edited reviewers, added: ilya-biryukov, hokein; removed: sammccall. sammccall added a comment. Sorry, I don't have any context on this (I just "blindly" fixed an asan error at one point). Adding original authors/reviewers of the test. Repository

[PATCH] D61697: [lit] Disable test on darwin when building shared libs.

2019-06-07 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land. LGTM, I see no way around it, we can't copy all of the shared libs. Maybe there's an alternative way to test this without copying the binary, but I don't have one in mind. Repos

[PATCH] D62804: [clangd] Enable extraction of system includes from custom toolchains

2019-06-07 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov added a comment. Could you give more context on what the custom toolchains are? One feasible alternative is to move this detection to clang's driver (where the rest of include path detection lives), why won't that work? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACT

[PATCH] D62998: [ARM] Fix bugs introduced by the fp64/d32 rework.

2019-06-07 Thread Simon Tatham via Phabricator via cfe-commits
simon_tatham created this revision. simon_tatham added reviewers: SjoerdMeijer, dmgreen, ostannard, samparker, JamesNagurne. Herald added subscribers: llvm-commits, cfe-commits, hiraditya, kristof.beyls, javed.absar, srhines. Herald added projects: clang, LLVM. Change D60691

[PATCH] D60710: [ARM] Add ACLE feature macros for MVE.

2019-06-07 Thread Sjoerd Meijer via Phabricator via cfe-commits
SjoerdMeijer updated this revision to Diff 203507. SjoerdMeijer added a comment. Added tests CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60710/new/ https://reviews.llvm.org/D60710 Files: clang/lib/Basic/Targets/ARM.cpp clang/lib/Basic/Targets/ARM.h clang/test/Preprocessor/arm-t

[PATCH] D62999: [clangd] Return 'RequestCancelled' on spurious completion triggers

2019-06-07 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov created this revision. ilya-biryukov added a reviewer: sammccall. Herald added subscribers: kadircet, arphaman, jkorous, MaskRay. Herald added a project: clang. We currently return 'UnknownErrorCode' and it causes `coc.nvim` to show an error message to the user, which we want to avoi

r362781 - [X86] -march=cooperlake (clang)

2019-06-07 Thread Pengfei Wang via cfe-commits
Author: pengfei Date: Fri Jun 7 01:53:37 2019 New Revision: 362781 URL: http://llvm.org/viewvc/llvm-project?rev=362781&view=rev Log: [X86] -march=cooperlake (clang) Support intel -march=cooperlake in clang Patch by Shengchen Kan (skan) Differential Revision: https://reviews.llvm.org/D62835 Mo

[PATCH] D62835: [X86] -march=cooperlake (clang)

2019-06-07 Thread Pengfei Wang via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL362781: [X86] -march=cooperlake (clang) (authored by pengfei, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D62

[PATCH] D62998: [ARM] Fix bugs introduced by the fp64/d32 rework.

2019-06-07 Thread Oliver Stannard (Linaro) via Phabricator via cfe-commits
ostannard accepted this revision. ostannard added a comment. This revision is now accepted and ready to land. LGTM with one change. Comment at: clang/lib/Driver/ToolChains/Arch/ARM.cpp:294 + std::vector FeaturesAfter; + You explained it in the commit message

[PATCH] D60710: [ARM] Add ACLE feature macros for MVE.

2019-06-07 Thread Oliver Stannard (Linaro) via Phabricator via cfe-commits
ostannard added inline comments. Comment at: clang/lib/Basic/Targets/ARM.cpp:465 + MVE |= MVE_INT | MVE_FP; + HW_FP |= HW_FP_SP | HW_FP_HP; } Does this also need to set FPU and HasLegalHalfType? CHANGES SINCE LAST ACTION https://reviews.llvm.or

[PATCH] D62998: [ARM] Fix bugs introduced by the fp64/d32 rework.

2019-06-07 Thread Sjoerd Meijer via Phabricator via cfe-commits
SjoerdMeijer added inline comments. Comment at: clang/lib/Driver/ToolChains/Arch/ARM.cpp:103 std::vector &Features, + std::vector &FeaturesAfter, const llvm::Triple &Triple) {

[PATCH] D62958: [clang-tidy] Fix descriptions for modernize-make-unique/shared checks.

2019-06-07 Thread Haojian Wu via Phabricator via cfe-commits
hokein abandoned this revision. hokein added a comment. As discussed offline, we don't need a separate fix description for this check, as the check warning message clearly implies what the fix will do. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D

[PATCH] D62999: [clangd] Return 'RequestCancelled' on spurious completion triggers

2019-06-07 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:734 +"ignored auto-triggered completion, preceding char did not match", +ErrorCode::Request

[PATCH] D62998: [ARM] Fix bugs introduced by the fp64/d32 rework.

2019-06-07 Thread Simon Tatham via Phabricator via cfe-commits
simon_tatham marked 2 inline comments as done. simon_tatham added inline comments. Comment at: clang/lib/Driver/ToolChains/Arch/ARM.cpp:449 //as well. -for (std::string Feature : {"vfp2", "vfp3", "vfp4", "fp-armv8", "fullfp16", -"n

[PATCH] D62962: Clang implementation of sizeless types

2019-06-07 Thread JinGu Kang via Phabricator via cfe-commits
jaykang10 added inline comments. Comment at: test/Sema/sizeless-1.c:29 + +typedef svint8_t vec_int8 __attribute__((vector_size(64))); // expected-error {{invalid vector element type}} + Please check "__attribute__((ext_vector_type()))". I guess it is also invali

r362785 - [CodeComplete] Improve overload handling for C++ qualified and ref-qualified methods.

2019-06-07 Thread Sam McCall via cfe-commits
Author: sammccall Date: Fri Jun 7 02:45:17 2019 New Revision: 362785 URL: http://llvm.org/viewvc/llvm-project?rev=362785&view=rev Log: [CodeComplete] Improve overload handling for C++ qualified and ref-qualified methods. Summary: - when a method is not available because of the target value kind

[PATCH] D62582: [CodeComplete] Improve overload handling for C++ qualified and ref-qualified methods.

2019-06-07 Thread Sam McCall via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. sammccall marked an inline comment as done. Closed by commit rL362785: [CodeComplete] Improve overload handling for C++ qualified and ref-qualified… (authored by sammccall, committed by ). Herald added a project: LLVM. Heral

r362786 - [X86][test] Add test cases using immediates to builtins-x86.c

2019-06-07 Thread Russell Gallop via cfe-commits
Author: russell_gallop Date: Fri Jun 7 02:51:44 2019 New Revision: 362786 URL: http://llvm.org/viewvc/llvm-project?rev=362786&view=rev Log: [X86][test] Add test cases using immediates to builtins-x86.c These builtins should work with immediate or variable shift operand for gcc compatibility. Di

[PATCH] D62850: [X86] Fix builtins-x86.c test where it wasn't using immediates. NFC

2019-06-07 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL362786: [X86][test] Add test cases using immediates to builtins-x86.c (authored by russell_gallop, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to c

[PATCH] D60710: [ARM] Add ACLE feature macros for MVE.

2019-06-07 Thread Sjoerd Meijer via Phabricator via cfe-commits
SjoerdMeijer marked an inline comment as done. SjoerdMeijer added inline comments. Comment at: clang/lib/Basic/Targets/ARM.cpp:465 + MVE |= MVE_INT | MVE_FP; + HW_FP |= HW_FP_SP | HW_FP_HP; } ostannard wrote: > Does this also need to set FPU and Ha

[PATCH] D60710: [ARM] Add ACLE feature macros for MVE.

2019-06-07 Thread Sjoerd Meijer via Phabricator via cfe-commits
SjoerdMeijer updated this revision to Diff 203522. SjoerdMeijer added a comment. Set FullFP16 for +mve.fp CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60710/new/ https://reviews.llvm.org/D60710 Files: clang/lib/Basic/Targets/ARM.cpp clang/lib/Basic/Targets/ARM.h clang/test/Prepr

[PATCH] D62808: [clangd] Print the ignored variant symbols

2019-06-07 Thread Haojian Wu via Phabricator via cfe-commits
hokein marked an inline comment as done. hokein added inline comments. Comment at: clang-tools-extra/clangd/include-mapping/gen_std.py:119 +( a list of tuple (symbol_name, relative_path_to_symbol_page), + a list of tuple (variant_symbol_name, caption_text) ) """ -

[PATCH] D50147: clang-format: support external styles

2019-06-07 Thread Francois Ferrand via Phabricator via cfe-commits
Typz added a comment. In D50147#157 , @sammccall wrote: > One thing that's unclear to me is whether your aim is to > > 1. solve a concrete problem for your organization > 2. solve a family of problems for similar organizations > 3. add a new way of con

[PATCH] D62998: [ARM] Fix bugs introduced by the fp64/d32 rework.

2019-06-07 Thread Simon Tatham via Phabricator via cfe-commits
simon_tatham updated this revision to Diff 203526. simon_tatham added a comment. Renamed `FeaturesAfter` to `ExtensionFeatures` and added a comment explaining why it has to be separate. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62998/new/ https

[PATCH] D62998: [ARM] Fix bugs introduced by the fp64/d32 rework.

2019-06-07 Thread Sjoerd Meijer via Phabricator via cfe-commits
SjoerdMeijer added a comment. Cheers, LGTM too. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62998/new/ https://reviews.llvm.org/D62998 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://

[PATCH] D61914: [Support][Test] Time profiler: add regression test

2019-06-07 Thread Anton Afanasyev via Phabricator via cfe-commits
anton-afanasyev marked 4 inline comments as done. anton-afanasyev added a comment. In D61914#1512021 , @aganea wrote: > Could you please move the test to a more approriate location? (ie. > clang/trunk/test/Driver/) Thanks, I've moved it there. ==

[PATCH] D61914: [Support][Test] Time profiler: add regression test

2019-06-07 Thread Anton Afanasyev via Phabricator via cfe-commits
anton-afanasyev updated this revision to Diff 203532. anton-afanasyev marked 2 inline comments as done. anton-afanasyev added a comment. Updated Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61914/new/ https://reviews.llvm.org/D61914 Files: clan

[PATCH] D62814: [clangd] Treat lambdas as functions when preparing hover response

2019-06-07 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet updated this revision to Diff 203533. kadircet marked 3 inline comments as done. kadircet added a comment. - Address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62814/new/ https://reviews.llvm.org/D62814 Files: clang-tools-ex

[PATCH] D62814: [clangd] Treat lambdas as functions when preparing hover response

2019-06-07 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet added inline comments. Comment at: clang-tools-extra/clangd/unittests/XRefsTests.cpp:831 + }; return HI; }}, ilya-biryukov wrote: > kadircet wrote: > > ilya-biryukov wrote: > > > kadircet wrote: > > > > ilya-biryukov wrote: > >

[PATCH] D60710: [ARM] Add ACLE feature macros for MVE.

2019-06-07 Thread Oliver Stannard (Linaro) via Phabricator via cfe-commits
ostannard added inline comments. Comment at: clang/lib/Basic/Targets/ARM.cpp:465 + MVE |= MVE_INT | MVE_FP; + HW_FP |= HW_FP_SP | HW_FP_HP; } SjoerdMeijer wrote: > ostannard wrote: > > Does this also need to set FPU and HasLegalHalfType? > Yep, tha

r362791 - [ARM] Fix bugs introduced by the fp64/d32 rework.

2019-06-07 Thread Simon Tatham via cfe-commits
Author: statham Date: Fri Jun 7 05:42:54 2019 New Revision: 362791 URL: http://llvm.org/viewvc/llvm-project?rev=362791&view=rev Log: [ARM] Fix bugs introduced by the fp64/d32 rework. Change D60691 caused some knock-on failures that weren't caught by the existing tests. Firstly, selecting a CPU t

[PATCH] D62998: [ARM] Fix bugs introduced by the fp64/d32 rework.

2019-06-07 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL362791: [ARM] Fix bugs introduced by the fp64/d32 rework. (authored by statham, committed by ). Herald added a subscriber: kristina. Changed prior to commit: https://reviews.llvm.org/D62998?vs=203526&id

[PATCH] D61914: [Support][Test] Time profiler: add regression test

2019-06-07 Thread Alexandre Ganea via Phabricator via cfe-commits
aganea accepted this revision. aganea added a comment. This revision is now accepted and ready to land. Thanks Anton! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61914/new/ https://reviews.llvm.org/D61914 __

r362792 - [Support][Test] Time profiler: add regression test

2019-06-07 Thread Anton Afanasyev via cfe-commits
Author: anton-afanasyev Date: Fri Jun 7 05:59:32 2019 New Revision: 362792 URL: http://llvm.org/viewvc/llvm-project?rev=362792&view=rev Log: [Support][Test] Time profiler: add regression test Summary: Add output to `llvm::errs()` when `-ftime-trace` option is enabled, add regression test checkin

[PATCH] D61914: [Support][Test] Time profiler: add regression test

2019-06-07 Thread Anton Afanasyev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL362792: [Support][Test] Time profiler: add regression test (authored by anton-afanasyev, committed by ). Changed prior to commit: https://reviews.llvm.org/D61914?vs=203532&id=203544#toc Repository: r

[PATCH] D62839: [clangd] Index API and implementations for relations

2019-06-07 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet marked an inline comment as done. kadircet added inline comments. Comment at: clang-tools-extra/clangd/index/FileIndex.cpp:202 + for (const auto &RelationSlab : RelationSlabs) { +for (const auto &R : *RelationSlab) { + AllRelations.push_back(R); ---

[PATCH] D62962: Clang implementation of sizeless types

2019-06-07 Thread Richard Sandiford via Phabricator via cfe-commits
rsandifo-arm updated this revision to Diff 203546. rsandifo-arm added a comment. Add tests for ext_vector_type Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62962/new/ https://reviews.llvm.org/D62962 Files: docs/SizelessTypes.rst include/clang/AST/Expr.h i

[PATCH] D61637: [Syntax] Introduce syntax trees

2019-06-07 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. In D61637#1527727 , @ilya-biryukov wrote: > I've addressed most of the comments, except the naming ones. > We need a convention for naming the language nodes and names for composite > and leaf structural nodes. > > For "langua

[PATCH] D61790: [C++20] add Basic consteval specifier

2019-06-07 Thread Tyker via Phabricator via cfe-commits
Tyker added inline comments. Comment at: clang/lib/Parse/ParseDecl.cpp:2491 + if (DS.hasConstexprSpecifier() && DSC != DeclSpecContext::DSC_condition) { Diag(DS.getConstexprSpecLoc(), diag::err_typename_invalid_constexpr); DS.ClearConstexprSpec(); rsmi

[PATCH] D61790: [C++20] add Basic consteval specifier

2019-06-07 Thread Tyker via Phabricator via cfe-commits
Tyker updated this revision to Diff 203551. Tyker marked 9 inline comments as done. Tyker added a comment. fixed requested changes except some i commented upon. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61790/new/ https://reviews.llvm.org/D61790 Files: clang/include/clang/AST/Dec

[PATCH] D62009: [clang] perform semantic checking in constant context

2019-06-07 Thread Tyker via Phabricator via cfe-commits
Tyker added a comment. @rsmith ping. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62009/new/ https://reviews.llvm.org/D62009 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit

[PATCH] D62569: [PowerPC] [Clang] Port SSE2 intrinsics to PowerPC

2019-06-07 Thread Jinsong Ji via Phabricator via cfe-commits
jsji accepted this revision. jsji added a comment. This revision is now accepted and ready to land. LGTM. Thanks for porting. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62569/new/ https://reviews.llvm.org/D62569 ___ cfe-commits mailing l

[PATCH] D60710: [ARM] Add ACLE feature macros for MVE.

2019-06-07 Thread Sjoerd Meijer via Phabricator via cfe-commits
SjoerdMeijer updated this revision to Diff 203555. SjoerdMeijer added a comment. Yep, sorry, missed that one. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60710/new/ https://reviews.llvm.org/D60710 Files: clang/lib/Basic/Targets/ARM.cpp clang/lib/Basic/Targets/ARM.h clang/test/P

[PATCH] D60233: [clang-scan-deps] initial outline of the tool that runs preprocessor to find dependencies over a JSON compilation database

2019-06-07 Thread Alexandre Ganea via Phabricator via cfe-commits
aganea added inline comments. Comment at: clang/test/ClangScanDeps/Inputs/regular_cdb.json:4 + "directory": "DIR", + "command": "clang -c DIR/regular_cdb.cpp -IInputs -MD -MF DIR/regular_cdb.d", + "file": "DIR/regular_cdb.cpp" Is `-MD -MF` required for clang-s

[PATCH] D62696: AMDGPU: Use AMDGPU toolchain for other OSes

2019-06-07 Thread Yaxun Liu via Phabricator via cfe-commits
yaxunl accepted this revision. yaxunl added a comment. This revision is now accepted and ready to land. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62696/new/ https://reviews.llvm.org/D62696 ___ cfe-commits mailing list cfe-commits@l

[PATCH] D60710: [ARM] Add ACLE feature macros for MVE.

2019-06-07 Thread Oliver Stannard (Linaro) via Phabricator via cfe-commits
ostannard accepted this revision. ostannard added a comment. This revision is now accepted and ready to land. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60710/new/ https://reviews.llvm.org/D60710 ___ cfe-commits mailing list cfe-com

[PATCH] D62574: Initial draft of target-configurable address spaces.

2019-06-07 Thread Bevin Hansson via Phabricator via cfe-commits
ebevhan added a comment. >> I'll have to see if that's possible without breaking a few more interfaces, >> since you can throw around Qualifiers and check for compatibility without an >> ASTContext today. >> >>> I was just thinking about testing the new logic. Should we add something >>> like

[PATCH] D62574: Initial draft of target-configurable address spaces.

2019-06-07 Thread Bevin Hansson via Phabricator via cfe-commits
ebevhan updated this revision to Diff 203559. ebevhan added a comment. Replaced `compatiblyIncludes` and its dependents with ASTContext accessors instead. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62574/new/ https://reviews.llvm.org/D62574 Files: include/clang/AST/ASTContext.h

[PATCH] D60710: [ARM] Add ACLE feature macros for MVE.

2019-06-07 Thread Sjoerd Meijer via Phabricator via cfe-commits
SjoerdMeijer added a comment. Thanks for reviewing! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60710/new/ https://reviews.llvm.org/D60710 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/list

r362806 - [ARM] Add ACLE feature macros for MVE

2019-06-07 Thread Sjoerd Meijer via cfe-commits
Author: sjoerdmeijer Date: Fri Jun 7 08:20:56 2019 New Revision: 362806 URL: http://llvm.org/viewvc/llvm-project?rev=362806&view=rev Log: [ARM] Add ACLE feature macros for MVE If MVE is present at all, then the macro __ARM_FEATURE_MVE is defined to a value which has bit 0 set for integer MVE, an

[PATCH] D60710: [ARM] Add ACLE feature macros for MVE.

2019-06-07 Thread Sjoerd Meijer via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL362806: [ARM] Add ACLE feature macros for MVE (authored by SjoerdMeijer, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.l

[PATCH] D62533: Build with _XOPEN_SOURCE defined on AIX

2019-06-07 Thread David Tenty via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL362808: Build with _XOPEN_SOURCE defined on AIX (authored by daltenty, committed by ). Changed prior to commit: https://reviews.llvm.org/D62533?vs=202822&id=203563#toc Repository: rL LLVM CHANGES SI

[PATCH] D63009: [OpenMP] Add target task alloc function with device ID

2019-06-07 Thread Gheorghe-Teodor Bercea via Phabricator via cfe-commits
gtbercea created this revision. gtbercea added reviewers: ABataev, AlexEichenberger, caomhin. Herald added subscribers: cfe-commits, jdoerfert, guansong. Herald added a project: clang. Add a new call to Clang to perform task allocation for the target. Repository: rC Clang https://reviews.llvm

[PATCH] D62999: [clangd] Return empty results on spurious completion triggers

2019-06-07 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov marked 2 inline comments as done. ilya-biryukov added inline comments. Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:734 +"ignored auto-triggered completion, preceding char did not match", +ErrorCode::RequestCancelled)); Server->codeCom

[PATCH] D62999: [clangd] Return empty results on spurious completion triggers

2019-06-07 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov updated this revision to Diff 203567. ilya-biryukov added a comment. - Return empty results instead of an error Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62999/new/ https://reviews.llvm.org/D62999 Files: clang-tools-extra/clang

[clang-tools-extra] r362811 - [clangd] Return empty results on spurious completion triggers

2019-06-07 Thread Ilya Biryukov via cfe-commits
Author: ibiryukov Date: Fri Jun 7 09:24:38 2019 New Revision: 362811 URL: http://llvm.org/viewvc/llvm-project?rev=362811&view=rev Log: [clangd] Return empty results on spurious completion triggers Summary: We currently return an error, this causes `coc.nvim` and VSCode to show an error message i

[PATCH] D62999: [clangd] Return empty results on spurious completion triggers

2019-06-07 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL362811: [clangd] Return empty results on spurious completion triggers (authored by ibiryukov, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit

r362812 - AST Matchers tutorial requests to enable clang-tools-extra. NFC

2019-06-07 Thread Ilya Biryukov via cfe-commits
Author: ibiryukov Date: Fri Jun 7 09:31:06 2019 New Revision: 362812 URL: http://llvm.org/viewvc/llvm-project?rev=362812&view=rev Log: AST Matchers tutorial requests to enable clang-tools-extra. NFC Otherwise the examples do not build. Modified: cfe/trunk/docs/LibASTMatchersTutorial.rst Mo

[PATCH] D62814: [clangd] Treat lambdas as functions when preparing hover response

2019-06-07 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov marked an inline comment as done. ilya-biryukov added inline comments. Comment at: clang-tools-extra/clangd/unittests/XRefsTests.cpp:831 + }; return HI; }}, kadircet wrote: > ilya-biryukov wrote: > > kadircet wrote: > > > i

[PATCH] D60710: [ARM] Add ACLE feature macros for MVE.

2019-06-07 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. The test is failing on some bots, e.g. http://lab.llvm.org:8011/builders/clang-with-lto-ubuntu/builds/13430/steps/test-stage1-compiler/logs/stdio /home/buildslave/buildslave1a/clang-with-lto-ubuntu/llvm.src/tools/clang/test/Preprocessor/arm-target-features.c:762:20: e

r362814 - [ARM] Add ACLE feature macros for MVE.

2019-06-07 Thread David Green via cfe-commits
Author: dmgreen Date: Fri Jun 7 10:28:12 2019 New Revision: 362814 URL: http://llvm.org/viewvc/llvm-project?rev=362814&view=rev Log: [ARM] Add ACLE feature macros for MVE. Fixup uninitialised variable. Modified: cfe/trunk/lib/Basic/Targets/ARM.cpp Modified: cfe/trunk/lib/Basic/Targets/ARM.

[PATCH] D60710: [ARM] Add ACLE feature macros for MVE.

2019-06-07 Thread Dave Green via Phabricator via cfe-commits
dmgreen added a comment. Hello. Hopefully fixed in rC362814 ! Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60710/new/ https://reviews.llvm.org/D60710 ___ cfe-commits mailing list cf

[PATCH] D61790: [C++20] add Basic consteval specifier

2019-06-07 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Only minor comments remain (other than the `-Wc++17-compat` warning). In the interest of incremental progress, let's leave the `-Wc++17-compat` warning for a later patch; feel free to commit t

r362816 - Revert rL362792 : [Support][Test] Time profiler: add regression test

2019-06-07 Thread Simon Pilgrim via cfe-commits
Author: rksimon Date: Fri Jun 7 10:37:04 2019 New Revision: 362816 URL: http://llvm.org/viewvc/llvm-project?rev=362816&view=rev Log: Revert rL362792 : [Support][Test] Time profiler: add regression test Summary: Add output to `llvm::errs()` when `-ftime-trace` option is enabled, add regression te

r362821 - [Support][Test] Time profiler: add regression test

2019-06-07 Thread Anton Afanasyev via cfe-commits
Author: anton-afanasyev Date: Fri Jun 7 11:13:48 2019 New Revision: 362821 URL: http://llvm.org/viewvc/llvm-project?rev=362821&view=rev Log: [Support][Test] Time profiler: add regression test Summary: Add output to `llvm::errs()` when `-ftime-trace` option is enabled, add regression test checkin

[PATCH] D62399: [clang] Add storage for APValue in ConstantExpr

2019-06-07 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Looks like a good first step, thanks! Comment at: clang/include/clang/AST/ASTContext.h:2821-2823 + /// Adds an APValue that will be destructed during the destruction of the

[PATCH] D62009: [clang] perform semantic checking in constant context

2019-06-07 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Thanks, looks great. Comment at: clang/lib/Sema/SemaChecking.cpp:13941 /// +/// \param isConstantEvaluated wether the evalaution should be permormed in +/// constant context

r362824 - Revert "[Support][Test] Time profiler: add regression test"

2019-06-07 Thread Anton Afanasyev via cfe-commits
Author: anton-afanasyev Date: Fri Jun 7 11:35:58 2019 New Revision: 362824 URL: http://llvm.org/viewvc/llvm-project?rev=362824&view=rev Log: Revert "[Support][Test] Time profiler: add regression test" This reverts commit 44282a60c90fdded249d57d91b12c6c0907102ec. This breaks buildbot. Removed:

[PATCH] D62949: [analyzer][tests] Add normalize_plist to replace diff_plist

2019-06-07 Thread Hubert Tong via Phabricator via cfe-commits
hubert.reinterpretcast added a comment. In D62949#1533605 , @NoQ wrote: > Ok! I'll be happy to have this addressed incrementally. I think it should be safe to at least commit the pre-normalization directly first. I'll take a look, and update the patch i

[PATCH] D59963: [clang-tidy] Add a module for the Linux kernel.

2019-06-07 Thread Nick Desaulniers via Phabricator via cfe-commits
nickdesaulniers added inline comments. Comment at: clang-tools-extra/test/clang-tidy/linuxkernel-must-check-errs.c:6 +// Prototypes of the error functions. +void * __must_check ERR_PTR(long error); +long __must_check PTR_ERR(const void *ptr); Let's come up with

[PATCH] D62636: Driver, IRGen: Set partitions on GlobalValues according to -fsymbol-partition flag.

2019-06-07 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. LGTM Comment at: clang/include/clang/Driver/Options.td:938 HelpText<"Disable C++ static destructor registration">; +def fsymbol_partition_EQ : Joined<["-"], "fsymbol-parti

r362829 - Driver, IRGen: Set partitions on GlobalValues according to -fsymbol-partition flag.

2019-06-07 Thread Peter Collingbourne via cfe-commits
Author: pcc Date: Fri Jun 7 12:10:08 2019 New Revision: 362829 URL: http://llvm.org/viewvc/llvm-project?rev=362829&view=rev Log: Driver, IRGen: Set partitions on GlobalValues according to -fsymbol-partition flag. Differential Revision: https://reviews.llvm.org/D62636 Added: cfe/trunk/test/

[PATCH] D62636: Driver, IRGen: Set partitions on GlobalValues according to -fsymbol-partition flag.

2019-06-07 Thread Peter Collingbourne via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. pcc marked an inline comment as done. Closed by commit rL362829: Driver, IRGen: Set partitions on GlobalValues according to -fsymbol-partition… (authored by pcc, committed by ). Herald added a project: LLVM. Herald added a s

Re: r362785 - [CodeComplete] Improve overload handling for C++ qualified and ref-qualified methods.

2019-06-07 Thread Vlad Tsyrklevich via cfe-commits
This change caused LSan failures and has been reverted in r362830: http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/32809 On Fri, Jun 7, 2019 at 2:42 AM Sam McCall via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: sammccall > Date: Fri Jun 7 02:45:17 2019 > New R

r362830 - Revert "[CodeComplete] Improve overload handling for C++ qualified and ref-qualified methods."

2019-06-07 Thread Vlad Tsyrklevich via cfe-commits
Author: vlad.tsyrklevich Date: Fri Jun 7 12:18:30 2019 New Revision: 362830 URL: http://llvm.org/viewvc/llvm-project?rev=362830&view=rev Log: Revert "[CodeComplete] Improve overload handling for C++ qualified and ref-qualified methods." This reverts commit f1f6e0fc2468e9c120b22b939507c527d08b8e

[PATCH] D61914: [Support][Test] Time profiler: add regression test

2019-06-07 Thread Puyan Lotfi via Phabricator via cfe-commits
plotfi added a comment. cfe/trunk/test/Driver/check-time-trace.cpp appears to fail on Darwin. Did you mean to pass the target explicitly ? Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61914/new/ https://reviews.llvm.org/D61914

[PATCH] D61914: [Support][Test] Time profiler: add regression test

2019-06-07 Thread Puyan Lotfi via Phabricator via cfe-commits
plotfi added inline comments. Comment at: cfe/trunk/test/Driver/check-time-trace.cpp:1 +// RUN: %clangxx -ftime-trace %s 2>&1 | grep "Time trace json-file dumped to" \ +// RUN: | awk '{print $NF}' | xargs cat \ This test should probably have // REQUIRES: shell

[PATCH] D63020: [HIP] Fix visibility for 'extern' device variables.

2019-06-07 Thread Michael Liao via Phabricator via cfe-commits
hliao created this revision. hliao added a reviewer: yaxunl. Herald added subscribers: cfe-commits, nhaehnle, jvesely. Herald added a project: clang. - Fix a bug which misses the change for a variable to be set with target-specific attributes. Repository: rG LLVM Github Monorepo https://revi

[PATCH] D62962: Clang implementation of sizeless types

2019-06-07 Thread JF Bastien via Phabricator via cfe-commits
jfb added a comment. How do these types mangle? Comment at: test/Sema/sizeless-1.c:60 + (void)__atomic_is_lock_free(1, &local_int8); + (void)__atomic_always_lock_free(1, &local_int8); + What do you expect these to return? Repository: rC Clang CHANGES SIN

[PATCH] D63018: [X86] Attempt to make the Intel core CPU inheritance a little more readable and maintainable

2019-06-07 Thread Craig Topper via Phabricator via cfe-commits
craig.topper updated this revision to Diff 203597. craig.topper added a comment. Fix mistake in a comment CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63018/new/ https://reviews.llvm.org/D63018 Files: clang/lib/Basic/Targets/X86.cpp Index: clang/lib/Basic/Targets/X86.cpp =

r362836 - Relax test so that the clang binary doesn't need to be named "clang".

2019-06-07 Thread Peter Collingbourne via cfe-commits
Author: pcc Date: Fri Jun 7 13:26:47 2019 New Revision: 362836 URL: http://llvm.org/viewvc/llvm-project?rev=362836&view=rev Log: Relax test so that the clang binary doesn't need to be named "clang". Modified: cfe/trunk/test/Driver/symbol-partition.c Modified: cfe/trunk/test/Driver/symbol-pa

[PATCH] D60974: Clang IFSO driver action.

2019-06-07 Thread Jordan Rupprecht via Phabricator via cfe-commits
rupprecht added a reviewer: rupprecht. rupprecht added a comment. Can you upload this patch with context? Either use arc or upload w/ -U9 I seem to have a lot of comments, but they're all nits -- my major concern being the `llvm_unreachable`s should be errors instead (i.e. should be trigger

[PATCH] D62885: [analyzer] Add werror flag for analyzer warnings

2019-06-07 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus accepted this revision. Szelethus added a comment. Comment at: include/clang/Driver/CC1Options.td:170 +def analyzer_werror : Flag<["-"], "analyzer-werror">, + HelpText<"Emit analyzer results as errors, not warnings">; + how about "rather than warnin

[PATCH] D61914: [Support][Test] Time profiler: add regression test

2019-06-07 Thread Anton Afanasyev via Phabricator via cfe-commits
anton-afanasyev marked an inline comment as done. anton-afanasyev added a comment. In D61914#1534508 , @plotfi wrote: > cfe/trunk/test/Driver/check-time-trace.cpp appears to fail on Darwin. Did you > mean to pass the target explicitly ? This commit is r

[PATCH] D63029: [NFC][CUDA] Avoid undefined grep in cuda-types.cu

2019-06-07 Thread Hubert Tong via Phabricator via cfe-commits
hubert.reinterpretcast created this revision. hubert.reinterpretcast added reviewers: jlebar, daltenty, xingxue, jasonliu. Herald added subscribers: jsji, jfb. Herald added a project: clang. vertical-line is not a BRE special character. POSIX.1-2017 XBD Section 9.3.2 indicates that the interpreta

[PATCH] D63029: [NFC][CUDA] Avoid undefined grep in cuda-types.cu

2019-06-07 Thread Artem Belevich via Phabricator via cfe-commits
tra added inline comments. Comment at: test/Preprocessor/cuda-types.cu:11 // RUN: %clang --cuda-host-only -nocudainc -target i386-unknown-linux-gnu -x cuda -E -dM -o - /dev/null \ -// RUN: | grep 'define __[^ ]*\(TYPE\|MAX\|SIZEOF|WIDTH\)\|define __GCC_ATOMIC' \ -// RUN: |

[PATCH] D63029: [NFC][CUDA] Avoid undefined grep in cuda-types.cu

2019-06-07 Thread Hubert Tong via Phabricator via cfe-commits
hubert.reinterpretcast added inline comments. Comment at: test/Preprocessor/cuda-types.cu:11 // RUN: %clang --cuda-host-only -nocudainc -target i386-unknown-linux-gnu -x cuda -E -dM -o - /dev/null \ -// RUN: | grep 'define __[^ ]*\(TYPE\|MAX\|SIZEOF|WIDTH\)\|define __GCC_ATO

[PATCH] D63030: [WebAssembly] Modernize include path handling

2019-06-07 Thread Sam Clegg via Phabricator via cfe-commits
sbc100 created this revision. Herald added subscribers: cfe-commits, sunfish, aheejin, jgravelle-google, dschuff. Herald added a project: clang. Move include path construction from InitHeaderSearch::AddDefaultIncludePaths in the Driver which appears to be the more modern/correct way of doing thin

[PATCH] D63030: [WebAssembly] Modernize include path handling

2019-06-07 Thread Sam Clegg via Phabricator via cfe-commits
sbc100 updated this revision to Diff 203634. sbc100 added a comment. - more OS Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63030/new/ https://reviews.llvm.org/D63030 Files: clang/lib/Driver/ToolChains/WebAssembly.cpp clang/lib/Frontend/InitH

[PATCH] D62926: [analyzer] ReturnVisitor: Bypass constructing objects to see inlined calls

2019-06-07 Thread Csaba Dabis via Phabricator via cfe-commits
Charusso updated this revision to Diff 203635. Charusso marked 2 inline comments as done. Charusso added a comment. - Added `CallExpr` as being purged away - More test CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62926/new/ https://reviews.llvm.org/D62926 Files: clang/lib/StaticAnal

[PATCH] D62926: [analyzer] ReturnVisitor: Bypass constructing objects to see inlined calls

2019-06-07 Thread Csaba Dabis via Phabricator via cfe-commits
Charusso added a comment. In D62926#1533560 , @NoQ wrote: > I'd like to have a test in which the constructor is //inlined//. What do you mean by that test? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62926/new/ https://reviews.llvm.org/D6292

[PATCH] D60974: Clang IFSO driver action.

2019-06-07 Thread Puyan Lotfi via Phabricator via cfe-commits
plotfi updated this revision to Diff 203638. plotfi added a comment. -U99 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60974/new/ https://reviews.llvm.org/D60974 Files: clang/include/clang/Driver/Options.td clang/include/clang/Driver/Type

[PATCH] D63031: DebugInfo: Render the canonical name of a class template specialization, even when nested in another class template specialization

2019-06-07 Thread David Blaikie via Phabricator via cfe-commits
dblaikie created this revision. dblaikie added a reviewer: rsmith. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rC Clang https://reviews.llvm.org/D63031 Files: lib/AST/TypePrinter.cpp lib/CodeGen/CGDebugInfo.cpp test/CodeGenCXX/debug-info-template-ex

LLVM buildmaster will be updated and restarted tonight

2019-06-07 Thread Galina Kistanova via cfe-commits
Hello everyone, LLVM buildmaster will be updated and restarted after 6PM Pacific time today. Thanks Galina ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r362855 - [analyzer] Add werror flag for analyzer warnings

2019-06-07 Thread Keno Fischer via cfe-commits
Author: kfischer Date: Fri Jun 7 16:34:00 2019 New Revision: 362855 URL: http://llvm.org/viewvc/llvm-project?rev=362855&view=rev Log: [analyzer] Add werror flag for analyzer warnings Summary: We're using the clang static analyzer together with a number of custom analyses in our CI system to ensu

[PATCH] D62885: [analyzer] Add werror flag for analyzer warnings

2019-06-07 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL362855: [analyzer] Add werror flag for analyzer warnings (authored by kfischer, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://re

r362856 - DebugInfo: Add support for 'nodebug' attribute on typedefs and alias templates

2019-06-07 Thread David Blaikie via cfe-commits
Author: dblaikie Date: Fri Jun 7 17:01:21 2019 New Revision: 362856 URL: http://llvm.org/viewvc/llvm-project?rev=362856&view=rev Log: DebugInfo: Add support for 'nodebug' attribute on typedefs and alias templates Seems like a logical extension to me - and of interest because it might help reduce

[PATCH] D63039: Various improvements to Clang MSVC Visualizers

2019-06-07 Thread Mike Spertus via Phabricator via cfe-commits
mspertus created this revision. mspertus added reviewers: aaron.ballman, zturner. mspertus added a project: clang. Herald added a subscriber: cfe-commits. This change adds/improves MSVC visualizers for many Clang types, including array types, trailing return types in function, deduction guides, a

[PATCH] D63029: [NFC][CUDA] Avoid undefined grep in cuda-types.cu

2019-06-07 Thread Hubert Tong via Phabricator via cfe-commits
hubert.reinterpretcast updated this revision to Diff 203659. hubert.reinterpretcast added a comment. Fix grep pattern in cuda-types.cu As requested, further fix so that the `SIZEOF` and `WIDTH` macros are checked. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D630

[PATCH] D63029: [CUDA] Fix grep pattern in cuda-types.cu

2019-06-07 Thread Hubert Tong via Phabricator via cfe-commits
hubert.reinterpretcast marked 3 inline comments as done. hubert.reinterpretcast added inline comments. Comment at: test/Preprocessor/cuda-types.cu:11 // RUN: %clang --cuda-host-only -nocudainc -target i386-unknown-linux-gnu -x cuda -E -dM -o - /dev/null \ -// RUN: | grep 'def

  1   2   >