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

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.

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

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.

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.

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

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)

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)

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.

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.

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.

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.

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

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.

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

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.

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.

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

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

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.

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.

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

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

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.

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.

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

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

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

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

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.

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.

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.

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

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

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

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

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

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

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

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

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

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

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.

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

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

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.

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.

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.

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.

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

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.

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

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

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

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

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

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

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

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

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.

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.

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.

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

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

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

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

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

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"

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

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.

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.

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.

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.

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.

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."

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

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

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.

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

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

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".

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.

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

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

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

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

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

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

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

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

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

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.

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

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

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

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

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

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

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

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

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   >