[PATCH] D55879: [X86][SSE] Auto upgrade PADDUS/PSUBUS intrinsics to UADD_SAT/USUB_SAT generic intrinsics (clang)

RKSimon created this revision. RKSimon added reviewers: craig.topper, spatel, nikic, tkrupa. Herald added a subscriber: llvm-commits. Sibling patch to D55855 , this emits UADD_SAT/USUB_SAT generic intrinsics for the SSE saturated math intrinsics instead of expand

[PATCH] D55823: [analyzer] Fix backward compatibility issue after D53280 'Emit an error for invalid -analyzer-config inputs'

Szelethus marked an inline comment as done. Szelethus added inline comments. Comment at: test/Analysis/invalid-analyzer-config-value.c:72 +// even if -analyze isn't specified. +// RUN: %clang --analyze -Xclang -analyzer-config -Xclang remember=TheVasa %s + NoQ wr

[PATCH] D55862: [Sema] Don't try to account for the size of an incomplete type in CheckArrayAccess

riccibruno updated this revision to Diff 178848. riccibruno marked 4 inline comments as done. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55862/new/ https://reviews.llvm.org/D55862 Files: lib/Sema/SemaChecking.cpp test/SemaCXX/array-bounds.cpp Index: test/

[PATCH] D55862: [Sema] Don't try to account for the size of an incomplete type in CheckArrayAccess

riccibruno added inline comments. Comment at: test/SemaCXX/array-bounds.cpp:294 + C &f() { return reinterpret_cast(xxx)[1]; } // no-warning + C &g() { return reinterpret_cast(xxx)[2]; } // expected-warning {{array index 2 is past the end of the array (which contains 2 elements

[PATCH] D54334: [AVR] Automatically link CRT and libgcc from the system avr-gcc

dylanmckay marked 3 inline comments as done. dylanmckay added a comment. > I'm not certain if it will be possible to devise test cases for the two > diagnostics I pointed out or not I don't think it will be. The compile warning logic works by directly querying the physical filesystem using the

[PATCH] D55823: [analyzer] Fix backward compatibility issue after D53280 'Emit an error for invalid -analyzer-config inputs'

Szelethus added a comment. Yea, should've used `hasArg` anyways. I'll look into this tonight. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55823/new/ https://reviews.llvm.org/D55823 ___ cfe-commits mailing list cfe-

[PATCH] D55818: [clangd] Unify path canonicalizations in the codebase

ilya-biryukov accepted this revision. ilya-biryukov added inline comments. This revision is now accepted and ready to land. Comment at: clangd/SourceCode.cpp:203 +FilePath)) { + elog("Could not turn relative path to absolute: {0}: {1}", FilePath, +

[PATCH] D55793: [clang-tidy] Add duplicated access specifier readability check (PR25403)

riccibruno added a comment. > Yes, I was thinking about the same thing! However, I believe that some people > may find this kind of "redundant" access specs actually more readable, so > maybe it makes sense to add a check option for this > to allow users to > disable it? Perhaps ? I will leave

[PATCH] D55818: [clangd] Unify path canonicalizations in the codebase

kadircet updated this revision to Diff 178856. kadircet marked 3 inline comments as done. kadircet added a comment. - Address comments Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55818/new/ https://reviews.llvm.org/D55818 Files: clangd/SourceCo

[clang-tools-extra] r349618 - [clangd] Unify path canonicalizations in the codebase

Author: kadircet Date: Wed Dec 19 02:46:21 2018 New Revision: 349618 URL: http://llvm.org/viewvc/llvm-project?rev=349618&view=rev Log: [clangd] Unify path canonicalizations in the codebase Summary: There were a few different places where we canonicalized paths, each one had its own flavor. This p

[PATCH] D55818: [clangd] Unify path canonicalizations in the codebase

This revision was automatically updated to reflect the committed changes. Closed by commit rL349618: [clangd] Unify path canonicalizations in the codebase (authored by kadircet, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM CHANGES SINCE LAST ACTION https://rev

[PATCH] D55848: [clang-tidy] Add export-fixes flag to clang-tidy-diff

hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land. looks good. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55848/new/ https://reviews.llvm.org/D55848 _

[PATCH] D55793: [clang-tidy] Add duplicated access specifier readability check (PR25403)

aaron.ballman added a comment. In D55793#1335243 , @m4tx wrote: > In D55793#1333723 , @riccibruno > wrote: > > > In D55793#1333691 , @m4tx wrote: > > > > > Don't use `CXXRec

[clang-tools-extra] r349623 - [clang-tidy] use "const SourceManager&" parameter, NFC.

Author: hokein Date: Wed Dec 19 05:25:08 2018 New Revision: 349623 URL: http://llvm.org/viewvc/llvm-project?rev=349623&view=rev Log: [clang-tidy] use "const SourceManager&" parameter, NFC. Modified: clang-tools-extra/trunk/clang-tidy/ClangTidyDiagnosticConsumer.cpp Modified: clang-tools-extr

[PATCH] D55885: [CodeComplete] Properly determine qualifiers of 'this' in a lambda

ilya-biryukov created this revision. ilya-biryukov added a reviewer: kadircet. The clang used to pick up the qualifiers of the lamba's call operator (which is always const) and fail to show non-const methods of 'this' in completion results. Repository: rC Clang https://reviews.llvm.org/D55885

[PATCH] D55885: [CodeComplete] Properly determine qualifiers of 'this' in a lambda

kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land. LGTM, thanks! Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55885/new/ https://reviews.llvm.org/D55885 ___ c

r349626 - [Index] Index paremeters in lambda expressions.

Author: hokein Date: Wed Dec 19 05:44:41 2018 New Revision: 349626 URL: http://llvm.org/viewvc/llvm-project?rev=349626&view=rev Log: [Index] Index paremeters in lambda expressions. Summary: This fixes clangd couldn't find references for lambda parameters. Reviewers: ilya-biryukov Subscribers: i

[PATCH] D55437: [Index] Index declarations in lambda expression.

This revision was automatically updated to reflect the committed changes. Closed by commit rC349626: [Index] Index paremeters in lambda expressions. (authored by hokein, committed by ). Changed prior to commit: https://reviews.llvm.org/D55437?vs=178688&id=178870#toc Repository: rC Clang CHA

r349627 - Portable Python script across Python version

Author: serge_sans_paille Date: Wed Dec 19 05:46:13 2018 New Revision: 349627 URL: http://llvm.org/viewvc/llvm-project?rev=349627&view=rev Log: Portable Python script across Python version urllib2 as been renamed into urllib and the library layout has changed. Workaround that in a consistent mann

[PATCH] D55199: Python2/3 compat - urllib

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rC349627: Portable Python script across Python version (authored by serge_sans_paille, committed by ). Herald added a subscr

[PATCH] D55628: Add support for "labels" on push/pop directives in #pragma clang attribute

aaron.ballman added a comment. In D55628#1335841 , @erik.pilkington wrote: > After looking through some users of `#pragma clang attribute`, I don't think > that the begin/end solution is what we want here. Some users of this > attribute write macros tha

[PATCH] D55784: [clang-tidy] Update abseil-duration-comparison to handle macro arguments

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. In D55784#1334929 , @hwright wrote: > @aaron.ballman I am both grateful and sad that I don't possess the same macro > creativity as you d

[PATCH] D55879: [X86][SSE] Auto upgrade PADDUS/PSUBUS intrinsics to UADD_SAT/USUB_SAT generic intrinsics (clang)

spatel accepted this revision. spatel added a comment. This revision is now accepted and ready to land. LGTM - I haven't looked in a while to see if it works, but I thought we have a script to auto-generate the CHECK lines for C-->IR tests, so we wouldn't need to 'CHECK-NOT' in these functions.

[PATCH] D55861: [OpenMP] Fix data sharing analysis in nested clause

ABataev accepted this revision. ABataev added a comment. This revision is now accepted and ready to land. LG Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55861/new/ https://reviews.llvm.org/D55861 ___ cfe-commits ma

[PATCH] D55865: [ObjC] Add a new attribute to opt-out of implicit callee retain/release in ARC

aaron.ballman added inline comments. Comment at: clang/include/clang/AST/Decl.h:1357 + bool isARCPseudoStrong() const { return VarDeclBits.ARCPseudoStrong; } + void setARCPseudoStrong(bool ps) { VarDeclBits.ARCPseudoStrong = ps; } Can you update `ps` to be `P

r349631 - [X86][SSE] Auto upgrade PADDUS/PSUBUS intrinsics to UADD_SAT/USUB_SAT generic intrinsics (clang)

Author: rksimon Date: Wed Dec 19 06:43:47 2018 New Revision: 349631 URL: http://llvm.org/viewvc/llvm-project?rev=349631&view=rev Log: [X86][SSE] Auto upgrade PADDUS/PSUBUS intrinsics to UADD_SAT/USUB_SAT generic intrinsics (clang) Sibling patch to D55855, this emits UADD_SAT/USUB_SAT generic int

[PATCH] D55879: [X86][SSE] Auto upgrade PADDUS/PSUBUS intrinsics to UADD_SAT/USUB_SAT generic intrinsics (clang)

This revision was automatically updated to reflect the committed changes. Closed by commit rC349631: [X86][SSE] Auto upgrade PADDUS/PSUBUS intrinsics to UADD_SAT/USUB_SAT generic… (authored by RKSimon, committed by ). Changed prior to commit: https://reviews.llvm.org/D55879?vs=178839&id=178880#

[PATCH] D55552: [Sema] Better static assert diagnostics for expressions involving temporaries.

courbet updated this revision to Diff 178882. courbet marked 2 inline comments as done. courbet added a comment. move PrintQualifiedTypes to PrintingPolicy Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D2/new/ https://reviews.llvm.org/D2 Files: include/

[PATCH] D55552: [Sema] Better static assert diagnostics for expressions involving temporaries.

courbet added inline comments. Comment at: include/clang/AST/Stmt.h:847 + const ASTContext *Context = nullptr, + bool PrintCanonicalTypes = false) const; aaron.ballman wrote: > I'm pretty wary of long lists of parameters that

[clang-tools-extra] r349632 - [clangd] Fix a syntax error on the test.

Author: hokein Date: Wed Dec 19 06:51:07 2018 New Revision: 349632 URL: http://llvm.org/viewvc/llvm-project?rev=349632&view=rev Log: [clangd] Fix a syntax error on the test. Modified: clang-tools-extra/trunk/unittests/clangd/FindSymbolsTests.cpp Modified: clang-tools-extra/trunk/unittests/cl

[PATCH] D55873: [analyzer] CStringChecker: Fix a crash when an argument of a weird type is encountered.

Szelethus accepted this revision. Szelethus added a comment. Cheers! Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55873/new/ https://reviews.llvm.org/D55873 ___ cfe-commits mailing list cfe-commits@lists.llvm.org ht

[PATCH] D55552: [Sema] Better static assert diagnostics for expressions involving temporaries.

aaron.ballman added inline comments. Comment at: lib/AST/StmtPrinter.cpp:78 const PrintingPolicy &Policy, unsigned Indentation = 0, -StringRef NL = "\n", -const ASTContext *Context = nullptr) +StringRef NL = "\n", c

[PATCH] D53928: Enable builtins necessary for SLEEF [AArch64] vectorized trigonometry libm functions

rengolin added reviewers: rsmith, chandlerc, rnk, ABataev. rengolin added a comment. Adding clang/omp developers for proper review. Please feel free to add more. FYI, there are four main ongoing discussions on the LLVM thread (D53927 ): 1. There is collusion bet

[PATCH] D53928: Enable builtins necessary for SLEEF [AArch64] vectorized trigonometry libm functions

steleman updated this revision to Diff 178884. steleman added a comment. Removed spurious patch for an unrelated change. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D53928/new/ https://reviews.llvm.org/D53928 Files: include/clang/Basic/Builtins.def include/

[PATCH] D55363: [clangd] Expose FileStatus in LSP.

ilya-biryukov added a comment. Could we add a capability to the init request to check for this extension? We don't want to send those notifications to clients that don't have the code to handle them. Another observation: when I played around with the previous version of the patch, `RunningActi

[PATCH] D55552: [Sema] Better static assert diagnostics for expressions involving temporaries.

courbet marked 6 inline comments as done. courbet added inline comments. Comment at: lib/AST/TypePrinter.cpp:165 +static SplitQualType splitAccordingToPolicy(QualType t, +const PrintingPolicy &Policy) { aaron.ballman

[PATCH] D55552: [Sema] Better static assert diagnostics for expressions involving temporaries.

courbet updated this revision to Diff 178887. courbet marked 2 inline comments as done. courbet added a comment. address review comments Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D2/new/ https://reviews.llvm.org/D2 Files: include/clang/AST/PrettyPri

[PATCH] D55552: [Sema] Better static assert diagnostics for expressions involving temporaries.

courbet updated this revision to Diff 17. courbet added a comment. clang-format diff Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D2/new/ https://reviews.llvm.org/D2 Files: include/clang/AST/PrettyPrinter.h include/clang/AST/Type.h lib/AST/Type

[PATCH] D55784: [clang-tidy] Update abseil-duration-comparison to handle macro arguments

hwright marked 2 inline comments as done. hwright added inline comments. Comment at: test/clang-tidy/abseil-duration-comparison.cpp:146 +#define VALUE_IF_2(e) (e) +#define VALUE_IF(v, e, type) (v ? VALUE_IF_2(absl::To##type##Seconds(e)) : 0) + int a3 = VALUE_IF(1, d1, Double); -

r349635 - [OpenMP] Fix data sharing analysis in nested clause

Author: jdenny Date: Wed Dec 19 07:59:47 2018 New Revision: 349635 URL: http://llvm.org/viewvc/llvm-project?rev=349635&view=rev Log: [OpenMP] Fix data sharing analysis in nested clause Without this patch, clang doesn't complain that X needs explicit data sharing attributes in the following: ```

[PATCH] D55861: [OpenMP] Fix data sharing analysis in nested clause

This revision was automatically updated to reflect the committed changes. Closed by commit rC349635: [OpenMP] Fix data sharing analysis in nested clause (authored by jdenny, committed by ). Changed prior to commit: https://reviews.llvm.org/D55861?vs=178803&id=178890#toc Repository: rC Clang

[clang-tools-extra] r349636 - [clang-tidy] Diagnose abseil-duration-comparison on macro arguments

Author: hwright Date: Wed Dec 19 08:03:34 2018 New Revision: 349636 URL: http://llvm.org/viewvc/llvm-project?rev=349636&view=rev Log: [clang-tidy] Diagnose abseil-duration-comparison on macro arguments Summary: This change relaxes the requirements on the utility `rewriteExprFromNumberToDuration`

[PATCH] D55784: [clang-tidy] Update abseil-duration-comparison to handle macro arguments

This revision was automatically updated to reflect the committed changes. hwright marked an inline comment as done. Closed by commit rL349636: [clang-tidy] Diagnose abseil-duration-comparison on macro arguments (authored by hwright, committed by ). Herald added a subscriber: llvm-commits. Changed

[PATCH] D54450: Get the correct range of tokens for preprocessor conditions

vmiklos added a comment. Ping, could this be reviewed, please? :-) Thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54450/new/ https://reviews.llvm.org/D54450 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.or

[PATCH] D55890: [X86][SSE] Auto upgrade PADDS/PSUBS intrinsics to SADD_SAT/SSUB_SAT generic intrinsics (clang)

RKSimon created this revision. RKSimon added reviewers: craig.topper, spatel, nikic, tkrupa. Herald added a subscriber: llvm-commits. This emits SADD_SAT/SSUB_SAT generic intrinsics for the SSE saturated math intrinsics. I'm still working on the llvm sibling patch (auto upgrade etc.) as there's

[PATCH] D55552: [Sema] Better static assert diagnostics for expressions involving temporaries.

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM, but you should wait a few days before committing in case @Quuxplusone has comments. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D2/new

[PATCH] D53891: [LTO] Add option to enable LTOUnit splitting, and disable unless needed

tejohnson marked 2 inline comments as done. tejohnson added inline comments. Comment at: lib/CodeGen/BackendUtil.cpp:831 + *OS, CodeGenOpts.EmitLLVMUseLists, EmitLTOSummary, + /*EmitModuleHash=*/false)); } pcc wrote: > Why add this argument

[PATCH] D55891: [compiler-rt] [xray] [tests] Detect and handle missing LLVMTestingSupport gracefully

mgorny created this revision. mgorny added reviewers: krytarowski, dberris, george.karpenkov. Herald added subscribers: Sanitizers, llvm-commits. Add a code to properly test for presence of LLVMTestingSupport library when performing a stand-alone build, and skip tests requiring it when it is not p

[PATCH] D55552: [Sema] Better static assert diagnostics for expressions involving temporaries.

Quuxplusone added a comment. LGTM. Tiny style suggestions, which I won't mind if you ignore. Comment at: include/clang/AST/Type.h:995 void getAsStringInternal(std::string &Str, - const PrintingPolicy &Policy) const { -return getAsStringInternal(

[PATCH] D55811: [compiler-rt] [sanitizer_common] Fix sha2 interceptors not to use vars in array len

This revision was automatically updated to reflect the committed changes. Closed by commit rL349645: [sanitizer_common] Fix sha2 interceptors not to use vars in array len (authored by mgorny, committed by ). Herald added a subscriber: delcypher. Changed prior to commit: https://reviews.llvm.org

r349648 - [Basic] Correct description of SanitizerSet.empty()

Author: mgorny Date: Wed Dec 19 09:25:51 2018 New Revision: 349648 URL: http://llvm.org/viewvc/llvm-project?rev=349648&view=rev Log: [Basic] Correct description of SanitizerSet.empty() Differential Revision: https://reviews.llvm.org/D55830 Modified: cfe/trunk/include/clang/Basic/Sanitizers.h

r349647 - [Driver] Disable -faddrsig by default on NetBSD

Author: mgorny Date: Wed Dec 19 09:25:46 2018 New Revision: 349647 URL: http://llvm.org/viewvc/llvm-project?rev=349647&view=rev Log: [Driver] Disable -faddrsig by default on NetBSD Avoid passing -faddrsig by default on NetBSD. This platform is still using old GNU binutils that crashes on executa

r349649 - [Driver] Add .hasAnySanitizer() to SanitizerArgs

Author: mgorny Date: Wed Dec 19 09:25:55 2018 New Revision: 349649 URL: http://llvm.org/viewvc/llvm-project?rev=349649&view=rev Log: [Driver] Add .hasAnySanitizer() to SanitizerArgs Add a simple method to query whether any sanitizer was enabled, via SanitizerArgs. This will be used in the NetBSD

r349650 - [Driver] [NetBSD] Add -D_REENTRANT when using sanitizers

Author: mgorny Date: Wed Dec 19 09:25:59 2018 New Revision: 349650 URL: http://llvm.org/viewvc/llvm-project?rev=349650&view=rev Log: [Driver] [NetBSD] Add -D_REENTRANT when using sanitizers NetBSD intends to support only reentrant interfaces in interceptors. When -lpthread is used without _REENTR

[PATCH] D55828: [clang] [Driver] Disable -faddrsig by default on NetBSD

This revision was automatically updated to reflect the committed changes. Closed by commit rC349647: [Driver] Disable -faddrsig by default on NetBSD (authored by mgorny, committed by ). Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55828/new/ https://reviews.llvm.

[PATCH] D55830: [clang] [Basic] Correct description of SanitizerSet.empty()

This revision was automatically updated to reflect the committed changes. Closed by commit rL349648: [Basic] Correct description of SanitizerSet.empty() (authored by mgorny, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D55830?vs=1786

[PATCH] D55832: [clang] [Driver] Add .hasAnySanitizer() to SanitizerArgs

This revision was automatically updated to reflect the committed changes. Closed by commit rL349649: [Driver] Add .hasAnySanitizer() to SanitizerArgs (authored by mgorny, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D55832?vs=178699&

[PATCH] D55654: [clang] [Driver] [NetBSD] Add -D_REENTRANT when using sanitizers

This revision was automatically updated to reflect the committed changes. mgorny marked an inline comment as done. Closed by commit rL349650: [Driver] [NetBSD] Add -D_REENTRANT when using sanitizers (authored by mgorny, committed by ). Herald added a subscriber: llvm-commits. Changed prior to com

[PATCH] D55850: [OpenCL] Allow address spaces as method qualifiers

Anastasia updated this revision to Diff 178904. Anastasia marked an inline comment as done. Anastasia added a comment. Address review comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55850/new/ https://reviews.llvm.org/D55850 Files: lib/Parse/ParseDecl.cpp lib/Sema/SemaOverl

[PATCH] D55850: [OpenCL] Allow address spaces as method qualifiers

Anastasia added inline comments. Comment at: lib/Parse/ParseDecl.cpp:6162 +} + } + rjmccall wrote: > This is enforcing a restriction that users write `const __private`, which > seems unreasonable. It looks like `ParseTypeQualifierList` takes a flag

[PATCH] D55850: [OpenCL] Allow address spaces as method qualifiers

Anastasia added a comment. In D55850#1335826 , @rjmccall wrote: > You're gating on OpenCL C++ in several places in this patch, but I don't > think you need to. This extension should be available to anyone using > address spaces and C++. Ok, I am happy

r349655 - [CodeComplete] Properly determine qualifiers of 'this' in a lambda

Author: ibiryukov Date: Wed Dec 19 10:01:24 2018 New Revision: 349655 URL: http://llvm.org/viewvc/llvm-project?rev=349655&view=rev Log: [CodeComplete] Properly determine qualifiers of 'this' in a lambda Summary: The clang used to pick up the qualifiers of the lamba's call operator (which is alway

[PATCH] D55885: [CodeComplete] Properly determine qualifiers of 'this' in a lambda

This revision was automatically updated to reflect the committed changes. Closed by commit rL349655: [CodeComplete] Properly determine qualifiers of 'this' in a lambda (authored by ibiryukov, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM CHANGES SINCE LAST ACTION

[PATCH] D55878: [Driver] Use --hash-style=gnu instead of both on FreeBSD

MaskRay updated this revision to Diff 178908. MaskRay edited the summary of this revision. MaskRay added a comment. Bring back MIPS special case Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55878/new/ https://reviews.llvm.org/D55878 Files: lib/Driver/ToolChai

r349657 - [OPENMP]Mark the loop as started when initialized.

Author: abataev Date: Wed Dec 19 10:16:37 2018 New Revision: 349657 URL: http://llvm.org/viewvc/llvm-project?rev=349657&view=rev Log: [OPENMP]Mark the loop as started when initialized. Need to mark the loop as started when the initialization statement is found. It is required to prevent possible

[PATCH] D55892: [OpenMP] 'close' map-type-modifier code generation

saghir created this revision. saghir added reviewers: ABataev, kkwli0, Hahnfeld, RaviNarayanaswamy, mikerice, hfinkel, gtbercea. saghir added a project: OpenMP. Herald added subscribers: cfe-commits, guansong. This patch provides codegen support for close map-type-modifier in map clause. A map c

[PATCH] D55892: [OpenMP] 'close' map-type-modifier code generation

ABataev accepted this revision. ABataev added a comment. This revision is now accepted and ready to land. LG Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55892/new/ https://reviews.llvm.org/D55892 ___ cfe-commits ma

[PATCH] D55847: [gn build] Add build file for clang/lib/Basic and dependencies

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55847/new/ https://reviews.llvm.org/D55847 ___ cfe-commits mailing list cfe-commits@l

[PATCH] D55850: [OpenCL] Allow address spaces as method qualifiers

rjmccall added inline comments. Comment at: lib/Parse/ParseDecl.cpp:6162 +} + } + Anastasia wrote: > rjmccall wrote: > > This is enforcing a restriction that users write `const __private`, which > > seems unreasonable. It looks like `ParseTypeQuali

LLVM lab maintenance works in the nearest hour

Hello everyone, LLVM lab could be unavailable for very short time in about next half hour due to maintenance works in llvm lab. Thank you for understanding. Thanks Galina ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llv

[PATCH] D55543: [CodeGen] Fix assertion on throwing object with inlined inherited constructor and non-trivial destructor.

vsapsai updated this revision to Diff 178921. vsapsai added a comment. - [ObjC++] Verify there are no unexpected calls. `--implicit-check-not` seems reasonable approach in this case. It causes adding `Inheritor` destructors but gives higher confidence there are no unexpected calls in unexpected p

[PATCH] D55823: [analyzer] Fix backward compatibility issue after D53280 'Emit an error for invalid -analyzer-config inputs'

Szelethus marked an inline comment as done. Szelethus added inline comments. Comment at: lib/Driver/ToolChains/Clang.cpp:3694 + { +auto AnalyzerConfigArg = When I dump `Args` for the following invocation, I get this: `clang -c dummie.cpp -Xclang -analyzer

[PATCH] D48753: [libcxx] Use custom allocator's `construct` in C++03 when available.

vsapsai added a comment. Ping. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D48753/new/ https://reviews.llvm.org/D48753 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D55895: NFC: simplify Darwin environment handling

jfb created this revision. jfb added reviewers: dexonsmith, arphaman. Herald added subscribers: cfe-commits, jkorous. The previous code detected conflicts through copy-pasta, this versions uses a 'loop'. Repository: rC Clang https://reviews.llvm.org/D55895 Files: lib/Driver/ToolChains/Darw

[PATCH] D55823: [analyzer] Fix backward compatibility issue after D53280 'Emit an error for invalid -analyzer-config inputs'

Szelethus updated this revision to Diff 178927. Szelethus added a comment. I'm still looking for a sensible solution, but I'll at least share a patch that actually works. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55823/new/ https://reviews.llvm.org/D55823 Files: lib/Driver/ToolC

[PATCH] D55823: [analyzer] Fix backward compatibility issue after D53280 'Emit an error for invalid -analyzer-config inputs'

Szelethus marked 2 inline comments as done. Szelethus added inline comments. Comment at: test/Analysis/invalid-analyzer-config-value.c:72 +// even if -analyze isn't specified. +// RUN: %clang --analyze -Xclang -analyzer-config -Xclang remember=TheVasa %s + Szelet

[PATCH] D55865: [ObjC] Add a new attribute to opt-out of implicit callee retain/release in ARC

erik.pilkington added inline comments. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:3492 +def warn_ignored_objc_externally_retained : Warning< + "'objc_externally_retained' can only be applied to strong retainable " + "object pointer types with automatic storage

[PATCH] D55865: [ObjC] Add a new attribute to opt-out of implicit callee retain/release in ARC

erik.pilkington updated this revision to Diff 178928. erik.pilkington marked 10 inline comments as done. erik.pilkington added a comment. Address @aaron.ballman comments, rebase onto r349535. Thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55865/new/ https://reviews.llvm.org/D5586

[PATCH] D55890: [X86][SSE] Auto upgrade PADDS/PSUBS intrinsics to SADD_SAT/SSUB_SAT generic intrinsics (clang)

craig.topper added inline comments. Comment at: lib/CodeGen/CGBuiltin.cpp:9492 static Value *EmitX86AddSubSatExpr(CodeGenFunction &CGF, - SmallVectorImpl &Ops, + SmallVectorImpl &Ops, bool IsSigned,

r349669 - PR40096: Forwards-compatible with C++20 rule regarding aggregates not having user-declared ctors

Author: dblaikie Date: Wed Dec 19 11:33:35 2018 New Revision: 349669 URL: http://llvm.org/viewvc/llvm-project?rev=349669&view=rev Log: PR40096: Forwards-compatible with C++20 rule regarding aggregates not having user-declared ctors Looks like these were in place to make these types move-only. Th

[PATCH] D55543: [CodeGen] Fix assertion on throwing object with inlined inherited constructor and non-trivial destructor.

rjmccall added a comment. Thanks, LGTM. Somewhat surprised that we don't have an implicit copy of the `Strong` argument, but it's not a bad thing that we don't. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55543/new/ https://reviews.llvm.org/D55543

[PATCH] D55862: [Sema] Don't try to account for the size of an incomplete type in CheckArrayAccess

efriedma added inline comments. Comment at: lib/Sema/SemaChecking.cpp:12357 + Context.getAsConstantArrayType(BaseExpr->getType()); + const Type *BaseType = BaseExpr->getType()->getPointeeOrArrayElementType(); + Using getPointeeOrArrayElementType here is kin

[PATCH] D55804: [analyzer] C++17: Fix leak false positives when an object with destructor is returned from the top frame.

NoQ updated this revision to Diff 178936. NoQ marked 2 inline comments as done. NoQ added a comment. Fxd! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55804/new/ https://reviews.llvm.org/D55804 Files: lib/StaticAnalyzer/Core/ExprEngineCXX.cpp test/Analysis/temporaries.cpp Index:

[PATCH] D55804: [analyzer] C++17: Fix leak false positives when an object with destructor is returned from the top frame.

NoQ added inline comments. Comment at: lib/StaticAnalyzer/Core/ExprEngineCXX.cpp:224 +bool IsArray = false; +V = makeZeroElementRegion(State, V, ReturnTy, IsArray); +assert(!IsArray && "Returning array from a function!"); dcoughlin wrote:

[PATCH] D55873: [analyzer] CStringChecker: Fix a crash when an argument of a weird type is encountered.

NoQ updated this revision to Diff 178938. NoQ marked an inline comment as done. NoQ added a comment. Indeed :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55873/new/ https://reviews.llvm.org/D55873 Files: lib/StaticAnalyzer/Checkers/CStringChecker.cpp test/Analysis/string.cpp I

[PATCH] D48753: [libcxx] Use custom allocator's `construct` in C++03 when available.

This revision was automatically updated to reflect the committed changes. Closed by commit rL349676: [libcxx] Use custom allocator's `construct` in C++03 when available. (authored by vsapsai, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.

[PATCH] D55878: [Driver] Use --hash-style=gnu instead of both on FreeBSD

sdardis added a comment. > I can't find rationale behind the MIPS discrepancy in the original commit. I > can add the if branch back if you tell me why... From my recollections, the gnu-hash style isn't supported in ld.bfd for MIPS. A patch was posted to the binutils list (https://sourceware.o

[PATCH] D55875: [analyzer] pr38668: RegionStore: Do not attempt to cast loaded values of non-scalar types.

NoQ marked an inline comment as done. NoQ added inline comments. Comment at: test/Analysis/bstring.cpp:47 + // The TRUE warning shows up on the path on which the vector is empty. clang_analyzer_eval(i == 66); // expected-warning {{UNKNOWN}} dcoughlin wrote:

[PATCH] D55847: [gn build] Add build file for clang/lib/Basic and dependencies

This revision was automatically updated to reflect the committed changes. Closed by commit rL349677: [gn build] Add build file for clang/lib/Basic and dependencies (authored by nico, committed by ). Herald added subscribers: llvm-commits, delcypher, mgorny. Changed prior to commit: https://revi

[PATCH] D52117: Generate llvm.loop.parallel_accesses instead of llvm.mem.parallel_loop_access metadata.

Meinersbur updated this revision to Diff 178944. Meinersbur added a comment. - Fix typo Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D52117/new/ https://reviews.llvm.org/D52117 Files: lib/CodeGen/CGLoopInfo.cpp lib/CodeGen/CGLoopInfo.h test/CodeGenCXX/prag

[PATCH] D55865: [ObjC] Add a new attribute to opt-out of implicit callee retain/release in ARC

aaron.ballman added inline comments. Comment at: clang/lib/Sema/SemaDeclAttr.cpp:6117 + // to ensure that the variable is 'const' so that we can error on + // modification, which can otherwise overrelease. + VD->setType(Ty.withConst()); erik.pilkington wrote:

[PATCH] D55865: [ObjC] Add a new attribute to opt-out of implicit callee retain/release in ARC

rjmccall added inline comments. Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:3492 +def warn_ignored_objc_externally_retained : Warning< + "'objc_externally_retained' can only be applied to strong retainable " + "object pointer types with automatic storage">, InG

Re: [PATCH] D55828: [clang] [Driver] Disable -faddrsig by default on NetBSD

On Tue, Dec 18, 2018 at 04:35:54PM +, Michał Górny via Phabricator via cfe-commits wrote: > mgorny created this revision. > mgorny added reviewers: krytarowski, pcc. > > Avoid passing -faddrsig by default on NetBSD. This platform is still > using old GNU binutils that crashes on executables

[PATCH] D55890: [X86][SSE] Auto upgrade PADDS/PSUBS intrinsics to SADD_SAT/SSUB_SAT generic intrinsics (clang)

RKSimon updated this revision to Diff 178946. RKSimon added a comment. Use ArrayRef Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55890/new/ https://reviews.llvm.org/D55890 Files: lib/CodeGen/CGBuiltin.cpp test/CodeGen/avx2-builtins.c test/CodeGen/avx512bw-

[PATCH] D55890: [X86][SSE] Auto upgrade PADDS/PSUBS intrinsics to SADD_SAT/SSUB_SAT generic intrinsics (clang)

craig.topper accepted this revision. craig.topper added a comment. This revision is now accepted and ready to land. LGTM Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55890/new/ https://reviews.llvm.org/D55890 ___ cfe

Re: [PATCH] D55828: [clang] [Driver] Disable -faddrsig by default on NetBSD

On Wed, 2018-12-19 at 21:51 +0100, Joerg Sonnenberger wrote: > On Tue, Dec 18, 2018 at 04:35:54PM +, Michał Górny via Phabricator via > cfe-commits wrote: > > mgorny created this revision. > > mgorny added reviewers: krytarowski, pcc. > > > > Avoid passing -faddrsig by default on NetBSD. Thi

[PATCH] D55873: [analyzer] CStringChecker: Fix a crash when an argument of a weird type is encountered.

NoQ added inline comments. Comment at: lib/StaticAnalyzer/Checkers/CStringChecker.cpp:2317-2320 else if (isCPPStdLibraryFunction(FDecl, "copy")) evalFunction = &CStringChecker::evalStdCopy; else if (isCPPStdLibraryFunction(FDecl, "copy_backward")) evalFunction =

[PATCH] D55895: NFC: simplify Darwin environment handling

arphaman added a comment. I think we might need to run it past the internal builds to see if anything breaks. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55895/new/ https://reviews.llvm.org/D55895 ___ cfe-commits

[PATCH] D55628: Add support for "labels" on push/pop directives in #pragma clang attribute

erik.pilkington updated this revision to Diff 178952. erik.pilkington marked 4 inline comments as done. erik.pilkington added a comment. Add Aaron's testcase, improve the documentation a bit. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55628/new/ https://reviews.llvm.org/D55628 Files

[PATCH] D55628: Add support for "labels" on push/pop directives in #pragma clang attribute

erik.pilkington added inline comments. Comment at: clang/lib/Parse/ParsePragma.cpp:3161 + if (!Tok.is(tok::period)) { +PP.Diag(Tok.getLocation(), diag::err_pragma_attribute_expected_period) +<< II; aaron.ballman wrote: > Can you reuse `di

  1   2   >