[PATCH] D55984: [gn build] Embed __TEXT __info_plist section into clang binary on macOS

This revision was automatically updated to reflect the committed changes. Closed by commit rL349992: [gn build] Embed __TEXT __info_plist section into clang binary on macOS (authored by nico, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

[PATCH] D55981: [gn build] Add build files for clang, clang-offload-bundler, and clang/lib/Headers

This revision was automatically updated to reflect the committed changes. Closed by commit rL349991: [gn build] Add build files for clang, clang-offload-bundler, and… (authored by nico, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

[PATCH] D55981: [gn build] Add build files for clang, clang-offload-bundler, and clang/lib/Headers

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/D55981/new/ https://reviews.llvm.org/D55981 ___ cfe-commits mailing list

[PATCH] D55981: [gn build] Add build files for clang, clang-offload-bundler, and clang/lib/Headers

thakis updated this revision to Diff 179404. thakis marked an inline comment as done. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55981/new/ https://reviews.llvm.org/D55981 Files: llvm/utils/gn/secondary/BUILD.gn llvm/utils/gn/secondary/clang/lib/Headers/BUILD.gn

[PATCH] D55981: [gn build] Add build files for clang, clang-offload-bundler, and clang/lib/Headers

thakis marked 2 inline comments as done. thakis added inline comments. Comment at: llvm/utils/gn/secondary/clang/lib/Headers/BUILD.gn:6 + +files = [ + "adxintrin.h", phosek wrote: > Why not just inline this? This variable doesn't seem to be used from anywhere

[PATCH] D56044: [Driver] Support object files in addition to static and shared libraries in compiler-rt

phosek created this revision. phosek added a reviewer: echristo. Herald added subscribers: cfe-commits, atanasyan, jrtc27, dberris, sdardis. This change introduces support for object files in addition to static and shared libraries which were already supported which requires changing the type of

[PATCH] D56043: [Driver] Don't pass default value to getCompilerRTArgString

phosek created this revision. phosek added a reviewer: echristo. Herald added a subscriber: cfe-commits. Using static library is already a default. This makes it easier to change the parameter type in the future. Repository: rC Clang https://reviews.llvm.org/D56043 Files:

[PATCH] D55980: [gn build] Add build file for clang/lib/FrontendTool

This revision was automatically updated to reflect the committed changes. Closed by commit rL349988: [gn build] Add build file for clang/lib/FrontendTool (authored by nico, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

[PATCH] D55979: [gn build] Add build file for clang/lib/ARCMigrate

This revision was automatically updated to reflect the committed changes. Closed by commit rL349987: [gn build] Add build file for clang/lib/ARCMigrate (authored by nico, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

[PATCH] D55978: [gn build] Add build files for clang/lib/{ASTMatchers, CrossTU}, clang/lib/StaticAnalyzer/{Checkers, Core, Frontend}

This revision was automatically updated to reflect the committed changes. Closed by commit rL349986: [gn build] Add build files for clang/lib/{ASTMatchers,CrossTU}… (authored by nico, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

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

This revision was automatically updated to reflect the committed changes. Closed by commit rL349984: [analyzer] pr38668: Do not attempt to cast loaded integers to floats. (authored by dergachev, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

r349984 - [analyzer] pr38668: Do not attempt to cast loaded integers to floats.

Author: dergachev Date: Fri Dec 21 18:06:51 2018 New Revision: 349984 URL: http://llvm.org/viewvc/llvm-project?rev=349984=rev Log: [analyzer] pr38668: Do not attempt to cast loaded integers to floats. This patch is a different approach to landing the reverted r349701. It is expected to have the

[PATCH] D56042: [analyzer] pr38838, pr39976: Fix a crash on emitting diagnostics before destructor.

NoQ created this revision. NoQ added reviewers: dcoughlin, xazax.hun, george.karpenkov, a_sidorin, rnkovacs, mikhail.ramalho, baloghadamsoftware. Herald added subscribers: cfe-commits, dkrupp, donat.nagy, Szelethus, a.sidorin, szepet. All right guys, now this one's weird. This patch looks

[PATCH] D56033: [CUDA] Treat extern global variable shadows same as regular extern vars.

This revision was automatically updated to reflect the committed changes. Closed by commit rL349981: [CUDA] Treat extern global variable shadows same as regular extern vars. (authored by tra, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

r349981 - [CUDA] Treat extern global variable shadows same as regular extern vars.

Author: tra Date: Fri Dec 21 17:11:09 2018 New Revision: 349981 URL: http://llvm.org/viewvc/llvm-project?rev=349981=rev Log: [CUDA] Treat extern global variable shadows same as regular extern vars. This fixes compiler crash when we attempted to compile this code: extern __device__ int data;

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

NoQ updated this revision to Diff 179389. NoQ added a comment. Herald added a subscriber: jfb. Add a denote - express test. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55875/new/ https://reviews.llvm.org/D55875 Files: lib/StaticAnalyzer/Checkers/ExprInspectionChecker.cpp

[PATCH] D55853: Ignore ConstantExpr in IgnoreParens

rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. LGTM with `s/ConstantExpr/FullExpr/`. Comment at: clang/lib/AST/Expr.cpp:2550 } +if (ConstantExpr *CE = dyn_cast(E)) { + E = CE->getSubExpr();

[PATCH] D54565: Introduce `-Wctad` as a subgroup of `-Wc++14-compat`

rsmith added a comment. >> I think there is a case to be made for warning on this specific language >> feature, because using CTAD on class templates that weren't designed for it >> is dangerous and creates source compatibility problems for future changes to >> that library. > > Personally I

[PATCH] D56033: [CUDA] Treat extern global variable shadows same as regular extern vars.

tra updated this revision to Diff 179387. tra added a comment. Fixed a typo. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56033/new/ https://reviews.llvm.org/D56033 Files: clang/lib/CodeGen/CodeGenModule.cpp clang/test/CodeGenCUDA/device-stub.cu Index:

[PATCH] D56000: [compiler-rt] [xray] Disable alignas() for thread_local objects on NetBSD

dberris added inline comments. Comment at: lib/xray/xray_defs.h:22 +#if !SANITIZER_NETBSD +#define XRAY_TLS_ALIGNAS(x) alignas(x) krytarowski wrote: > I would switch the order, in order to remove unneeded negation. > > ``` > #if SANITIZER_NETBSD > ... > #else

[PATCH] D55853: Ignore ConstantExpr in IgnoreParens

rnk added a comment. Want to stamp this? It's 4pm on the Friday before Christmas, what could go wrong? :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55853/new/ https://reviews.llvm.org/D55853 ___ cfe-commits mailing list

[PATCH] D56033: [CUDA] Treat extern global variable shadows same as regular extern vars.

jlebar accepted this revision. jlebar added inline comments. This revision is now accepted and ready to land. Comment at: clang/test/CodeGenCUDA/device-stub.cu:51 +// external device-side variables with definitiions should generate +// definitions for the shadows.

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

NoQ updated this revision to Diff 179379. NoQ added a comment. Add a comment. I guess it doesn't really matter why does this lead to a crash. The symbol itself is well-formed, but we probably don't support it yet in some place, and hopefully (but not necessarily) `CastRetrievedVal` is the only

[PATCH] D56033: [CUDA] Treat extern global variable shadows same as regular extern vars.

tra created this revision. tra added a reviewer: jlebar. Herald added subscribers: bixia, sanjoy. This fixes compiler crash when we attempted to compile this code: extern __device__ int data; __device__ int data = 1; https://reviews.llvm.org/D56033 Files:

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

george.karpenkov accepted this revision. george.karpenkov added a comment. Seems good, but a comment explaining why this is necessary and why the crash follows otherwise would be great. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55875/new/ https://reviews.llvm.org/D55875

[PATCH] D56025: [clang-tidy] add IgnoreMacros option to readability-uppercase-literal-suffix

vmiklos marked 2 inline comments as done. vmiklos added inline comments. Comment at: test/clang-tidy/readability-uppercase-literal-suffix-integer-macro.cpp:2-3 +// RUN: %check_clang_tidy %s readability-uppercase-literal-suffix %t -- \ +// RUN: -config="{CheckOptions: [{key:

[PATCH] D56025: [clang-tidy] add IgnoreMacros option to readability-uppercase-literal-suffix

vmiklos updated this revision to Diff 179371. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56025/new/ https://reviews.llvm.org/D56025 Files: clang-tidy/readability/UppercaseLiteralSuffixCheck.cpp clang-tidy/readability/UppercaseLiteralSuffixCheck.h docs/ReleaseNotes.rst

[PATCH] D55955: Properly diagnose [[nodiscard]] on the body of a range-based for loop

rsmith added inline comments. Comment at: lib/Sema/SemaStmt.cpp:2846 diag::warn_empty_range_based_for_body); + DiagnoseUnusedExprResult(B); aaron.ballman wrote: > rsmith wrote: > > rsmith wrote: > > > While this looks correct per the

[PATCH] D55978: [gn build] Add build files for clang/lib/{ASTMatchers, CrossTU}, clang/lib/StaticAnalyzer/{Checkers, Core, Frontend}

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/D55978/new/ https://reviews.llvm.org/D55978 ___ cfe-commits mailing list

[PATCH] D55984: [gn build] Embed __TEXT __info_plist section into clang binary on macOS

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/D55984/new/ https://reviews.llvm.org/D55984 ___ cfe-commits mailing list

[PATCH] D55981: [gn build] Add build files for clang, clang-offload-bundler, and clang/lib/Headers

phosek added inline comments. Comment at: llvm/utils/gn/secondary/clang/lib/Headers/BUILD.gn:6 + +files = [ + "adxintrin.h", Why not just inline this? This variable doesn't seem to be used from anywhere else? CHANGES SINCE LAST ACTION

[PATCH] D55979: [gn build] Add build file for clang/lib/ARCMigrate

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/D55979/new/ https://reviews.llvm.org/D55979 ___ cfe-commits mailing list

[PATCH] D55980: [gn build] Add build file for clang/lib/FrontendTool

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/D55980/new/ https://reviews.llvm.org/D55980 ___ cfe-commits mailing list

[PATCH] D55490: Add dumpMethodDeclOverrides to NodeDumper

steveire abandoned this revision. steveire added a comment. Abandoning this because it's not really the right approach as it appears in the interface between the generic traverser and the generic node dumper, which it really shouldn't. Repository: rC Clang CHANGES SINCE LAST ACTION

[PATCH] D55489: Implement dumpFunctionDeclParameters in NodeDumper

steveire abandoned this revision. steveire added a comment. Abandoning this because it's not really the right approach as it appears in the interface between the generic traverser and the generic node dumper, which it really shouldn't. Repository: rC Clang CHANGES SINCE LAST ACTION

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

NoQ updated this revision to Diff 179355. NoQ added a comment. `// no-crash` and a slightly cleaner test. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55875/new/ https://reviews.llvm.org/D55875 Files: lib/StaticAnalyzer/Core/Store.cpp test/Analysis/casts.c

[PATCH] D56025: [clang-tidy] add IgnoreMacros option to readability-uppercase-literal-suffix

lebedev.ri added inline comments. Comment at: test/clang-tidy/readability-uppercase-literal-suffix-integer-macro.cpp:2-3 +// RUN: %check_clang_tidy %s readability-uppercase-literal-suffix %t -- \ +// RUN: -config="{CheckOptions: [{key:

[PATCH] D56025: [clang-tidy] add IgnoreMacros option to readability-uppercase-literal-suffix

vmiklos created this revision. vmiklos added reviewers: JonasToth, alexfh, lebedev.ri. Herald added subscribers: cfe-commits, xazax.hun. And also enable it by default to be consistent with e.g. modernize-use-using. This helps e.g. when running this check on client code where the macro is

[PATCH] D56024: [clang] [Distro] Support detecting Gentoo

phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM Comment at: include/clang/Driver/Distro.h:118 } bool IsUbuntu() const { Shall we also introduce the `IsGentoo()` predicate for convenience?

[PATCH] D56024: [clang] [Distro] Support detecting Gentoo

mgorny created this revision. mgorny added reviewers: sylvestre.ledru, bruno, bkramer, phosek. Add support for distinguishing plain Gentoo distribution, and a unit test for it. This is going to be used to introduce distro-specific customizations in the driver code; most notably, it is going to

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

NoQ updated this revision to Diff 179352. NoQ added a comment. Add test case `bool_to_nullptr` in `casts.cpp` on which my second attempt crashes but the current code does not. CHANGES SINCE LAST

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

MaskRay added a comment. In D55878#1339098 , @emaste wrote: > I think the arch-change (switching from a whitelist to a MIPS blacklist) is > reasonable. What is the motivation for dropping `DT_HASH`, just binary size > reduction? Yes. It saves a few

[PATCH] D55933: [Sema] Do not print default template parameters.

Quuxplusone added inline comments. Comment at: test/SemaCXX/static-assert-cxx17.cpp:109 +template +void foo7() { + static_assert(X()); courbet wrote: > Quuxplusone wrote: > > None of these new test cases actually involve the default template argument. > > > >

[PATCH] D56022: [AST] Store the arguments of CXXConstructExpr in a trailing array

rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. LGTM. Comment at: include/clang/AST/ExprCXX.h:1613 + + TypeSourceInfo *TSI = nullptr; + riccibruno wrote: > Changed the name from `Type` since `Type`

[PATCH] D54565: Introduce `-Wctad` as a subgroup of `-Wc++14-compat`

Quuxplusone added a comment. In D54565#1339454 , @thakis wrote: > Since I don't know what it means, I don't know if I want a diagnostic for it > :-) Fair point. ;) But the full text of the diagnostic message does say "class template argument deduction

[PATCH] D55710: add pragmas to control Software Pipelining optimisation

alexey.lapshin marked 2 inline comments as done. alexey.lapshin added a comment. will correct all mistakes. please check explanations for the questions. Comment at: include/clang/Basic/AttrDocs.td:2655 + dependence violated. This constant interval(in cycles) between the start

[PATCH] D56022: [AST] Store the arguments of CXXConstructExpr in a trailing array

riccibruno marked 2 inline comments as done. riccibruno added inline comments. Comment at: include/clang/AST/ExprCXX.h:1295 + /// A pointer to the constructor which will be ultimately called. + CXXConstructorDecl *Constructor; + Removed the default member

[PATCH] D56022: [AST] Store the arguments of CXXConstructExpr in a trailing array

riccibruno created this revision. riccibruno added a reviewer: rjmccall. riccibruno added a project: clang. Herald added a reviewer: shafik. Herald added a subscriber: cfe-commits. Store the arguments of `CXXConstructExpr` in a trailing array. This is very similar to the `CallExpr` case in

r349955 - Switch from static_cast<> to cast<>, update identifier for coding conventions; NFC.

Author: aaronballman Date: Fri Dec 21 13:11:36 2018 New Revision: 349955 URL: http://llvm.org/viewvc/llvm-project?rev=349955=rev Log: Switch from static_cast<> to cast<>, update identifier for coding conventions; NFC. Modified: cfe/trunk/lib/Sema/SemaStmt.cpp Modified:

[PATCH] D56012: [clang-tidy] Be more liberal about literal zeroes in abseil checks

This revision was automatically updated to reflect the committed changes. Closed by commit rL349953: [clang-tidy] Be more liberal about literal zeroes in abseil checks (authored by hwright, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

[clang-tools-extra] r349953 - [clang-tidy] Be more liberal about literal zeroes in abseil checks

Author: hwright Date: Fri Dec 21 13:07:11 2018 New Revision: 349953 URL: http://llvm.org/viewvc/llvm-project?rev=349953=rev Log: [clang-tidy] Be more liberal about literal zeroes in abseil checks Summary: Previously, we'd only match on literal floating or integral zeroes, but I've now also

[PATCH] D56012: [clang-tidy] Be more liberal about literal zeroes in abseil checks

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56012/new/ https://reviews.llvm.org/D56012 ___ cfe-commits mailing

[PATCH] D55869: Convert some ObjC retain/release msgSends to runtime calls

pete added a comment. In D55869#1339537 , @rjmccall wrote: > It sounds like it's fine. Thanks for the review! Just pushed it as r349952. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55869/new/

[PATCH] D55869: Convert some ObjC retain/release msgSends to runtime calls

This revision was automatically updated to reflect the committed changes. Closed by commit rC349952: Convert some ObjC retain/release msgSends to runtime calls. (authored by pete, committed by ). Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55869/new/

r349952 - Convert some ObjC retain/release msgSends to runtime calls.

Author: pete Date: Fri Dec 21 13:00:32 2018 New Revision: 349952 URL: http://llvm.org/viewvc/llvm-project?rev=349952=rev Log: Convert some ObjC retain/release msgSends to runtime calls. It is faster to directly call the ObjC runtime for methods such as retain/release instead of sending a

[PATCH] D56012: [clang-tidy] Be more liberal about literal zeroes in abseil checks

hwright added a comment. btw, I think `hasInit` should probably be moved into the core set of matchers at some point. Comment at: clang-tidy/abseil/DurationRewriter.cpp:110 + ast_matchers::internal::Matcher, InnerMatcher) { + return (N < Node.getNumInits() && +

[PATCH] D56012: [clang-tidy] Be more liberal about literal zeroes in abseil checks

hwright updated this revision to Diff 179335. hwright marked 6 inline comments as done. hwright added a comment. Add documentation, adjust test. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56012/new/ https://reviews.llvm.org/D56012 Files:

[PATCH] D55949: Correctly handle function pointers returning a type marked nodiscard

aaron.ballman requested review of this revision. aaron.ballman added a comment. There's enough churn based on the review feedback that this should probably have a second round of review just to be sure. Comment at: lib/AST/Expr.cpp:2281-2286 + // If there is no

r349949 - [Sema][NFC] Fix a Wimplicit-fallthrough warning in CheckSpecializationInstantiationRedecl

Author: brunoricci Date: Fri Dec 21 12:38:06 2018 New Revision: 349949 URL: http://llvm.org/viewvc/llvm-project?rev=349949=rev Log: [Sema][NFC] Fix a Wimplicit-fallthrough warning in CheckSpecializationInstantiationRedecl All cases are covered so add an llvm_unreachable. NFC. Modified:

[PATCH] D55869: Convert some ObjC retain/release msgSends to runtime calls

rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. It sounds like it's fine. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55869/new/ https://reviews.llvm.org/D55869 ___ cfe-commits

r349948 - [AST][NFC] Remove stale comment in CXXRecordDecl::is(Virtually)DerivedFrom.

Author: brunoricci Date: Fri Dec 21 12:23:07 2018 New Revision: 349948 URL: http://llvm.org/viewvc/llvm-project?rev=349948=rev Log: [AST][NFC] Remove stale comment in CXXRecordDecl::is(Virtually)DerivedFrom. The "this" capture was removed in r291939. Modified:

[PATCH] D56012: [clang-tidy] Be more liberal about literal zeroes in abseil checks

aaron.ballman added inline comments. Comment at: clang-tidy/abseil/DurationRewriter.cpp:110 + ast_matchers::internal::Matcher, InnerMatcher) { + return (N < Node.getNumInits() && + InnerMatcher.matches(*Node.getInit(N)->IgnoreParenImpCasts(), Finder,

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

NoQ closed this revision. NoQ added a comment. Should be sorted our in rC349866 . Ugh, what a mess :) Thanks everyone :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55823/new/ https://reviews.llvm.org/D55823

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

aaron.ballman added inline comments. Comment at: clang-tidy/readability/DuplicatedAccessSpecifiersCheck.cpp:51 + diag(ASDecl->getLocation(), "duplicated access specifier") + << MatchedDecl + << FixItHint::CreateRemoval(ASDecl->getSourceRange());

[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/test/SemaObjC/externally-retained.m:14 + + EXT_RET int (^d)() = ^{return 0;}; + EXT_RET ObjCTy *e = 0; erik.pilkington wrote: > aaron.ballman wrote: > > Should this be useful for function pointer

[PATCH] D55915: [Driver] Make -fno-omit-frame-pointer imply -mno-omit-leaf-frame-pointer

tabloid.adroit updated this revision to Diff 179325. tabloid.adroit added a comment. - [Driver] Make -fno-omit-frame-pointer imply -mno-omit-leaf-frame-pointer - add test - update Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55915/new/

r349943 - [analyzer] Tests quickfix.

Author: george.karpenkov Date: Fri Dec 21 11:40:44 2018 New Revision: 349943 URL: http://llvm.org/viewvc/llvm-project?rev=349943=rev Log: [analyzer] Tests quickfix. Modified: cfe/trunk/test/Analysis/osobject-retain-release.cpp Modified: cfe/trunk/test/Analysis/osobject-retain-release.cpp

[PATCH] D54565: Introduce `-Wctad` as a subgroup of `-Wc++14-compat`

thakis added a comment. In D54565#1331293 , @Quuxplusone wrote: > @rsmith ping! > > @thakis: You said, //"From a user point of view, I have no idea what "CTAD" > means, and I sometimes work on a C++ compiler."// Did you mean "I think the > command-line

[PATCH] D55710: add pragmas to control Software Pipelining optimisation

aaron.ballman added inline comments. Comment at: include/clang/Basic/AttrDocs.td:2650 +Software Pipelining optimization is a technique used to optimize loops by + utilizing instructions level parallelism. It reorders loop instructions to + overlap iterations. As the result

[PATCH] D55455: Remove stat cache chaining as it's no longer needed after PTH support has been removed

This revision was automatically updated to reflect the committed changes. Closed by commit rL349942: Remove stat cache chaining as its no longer needed after PTH support has been (authored by arphaman, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

r349942 - Remove stat cache chaining as it's no longer needed after PTH support has been

Author: arphaman Date: Fri Dec 21 11:33:09 2018 New Revision: 349942 URL: http://llvm.org/viewvc/llvm-project?rev=349942=rev Log: Remove stat cache chaining as it's no longer needed after PTH support has been removed Stat cache chaining was implemented for a StatListener in the PTH writer so

[PATCH] D55932: [Sema] Simplfy static_assert diagnostic code.

aaron.ballman added inline comments. Comment at: test/SemaCXX/static-assert.cpp:130 static_assert(std::is_same()), int>::value, "message"); -// expected-error@-1{{static_assert failed due to requirement 'std::is_same, int>::value' "message"}} +//

Re: [PATCH] D55792: Allow direct navigation to static analysis checker documentation through SARIF exports

On Fri, Dec 21, 2018 at 12:35 PM Phil Camp via Phabricator wrote: > > FlameTop added a comment. > > I'm afraid we are seeing a build failure here on our local Windows checking > MSVC build. Unfortunately I cannot find a public buildbot that uses the exact > configuration that causes the

r349940 - Switch from cast<> to dyn_cast<>.

Author: aaronballman Date: Fri Dec 21 11:16:38 2018 New Revision: 349940 URL: http://llvm.org/viewvc/llvm-project?rev=349940=rev Log: Switch from cast<> to dyn_cast<>. This avoids a potential failed assertion that is happening on someone's out-of-tree build. Modified:

r349938 - [analyzer] Correct the summary violation diagnostics for the retain count checker

Author: george.karpenkov Date: Fri Dec 21 11:13:28 2018 New Revision: 349938 URL: http://llvm.org/viewvc/llvm-project?rev=349938=rev Log: [analyzer] Correct the summary violation diagnostics for the retain count checker It should be in the past tense. Modified:

r349939 - Revert "Revert rL349876 from cfe/trunk: [analyzer] Perform escaping in RetainCountChecker on type mismatch even for inlined functions"

Author: george.karpenkov Date: Fri Dec 21 11:13:40 2018 New Revision: 349939 URL: http://llvm.org/viewvc/llvm-project?rev=349939=rev Log: Revert "Revert rL349876 from cfe/trunk: [analyzer] Perform escaping in RetainCountChecker on type mismatch even for inlined functions" This reverts commit

[PATCH] D55949: Correctly handle function pointers returning a type marked nodiscard

aaron.ballman updated this revision to Diff 179318. aaron.ballman marked 3 inline comments as done. aaron.ballman added a comment. Updated based on review feedback. Moved `FunctionDecl::hasUnusedResultAttr()` and `FunctionDecl::getUnusedResultAttr()` to `CallExpr` to reduce logic duplication

[PATCH] D56012: [clang-tidy] Be more liberal about literal zeroes in abseil checks

Eugene.Zelenko added inline comments. Comment at: test/clang-tidy/abseil-duration-factory-scale.cpp:3 +#include #include "absl/time/time.h" cstdint, please. Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION

[PATCH] D56012: Be more liberal about literal zeroes in abseil checks

hwright created this revision. hwright added reviewers: aaron.ballman, JonasToth, alexfh, hokein. hwright added a project: clang-tools-extra. Herald added a subscriber: cfe-commits. Previously, we'd only match on literal floating or integral zeroes, but I've now also learned that some users

r349934 - [AST][NFC] Fix Wsign-compare warning introduced in CXXOperatorCallExpr

Author: brunoricci Date: Fri Dec 21 09:54:51 2018 New Revision: 349934 URL: http://llvm.org/viewvc/llvm-project?rev=349934=rev Log: [AST][NFC] Fix Wsign-compare warning introduced in CXXOperatorCallExpr Modified: cfe/trunk/lib/AST/ExprCXX.cpp Modified: cfe/trunk/lib/AST/ExprCXX.cpp URL:

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

tejohnson updated this revision to Diff 179305. tejohnson added a comment. Implement comment suggestions. Added test for new error. Also test changes for LLVM side changes in D53890 . Repository: rC Clang CHANGES SINCE LAST ACTION

r349933 - [Sema][NFC] Fix Wimplicit-fallthrough warning in getCursorKindForDecl

Author: brunoricci Date: Fri Dec 21 09:52:13 2018 New Revision: 349933 URL: http://llvm.org/viewvc/llvm-project?rev=349933=rev Log: [Sema][NFC] Fix Wimplicit-fallthrough warning in getCursorKindForDecl All cases are covered so add an llvm_unreachable. NFC. Modified:

[PATCH] D55792: Allow direct navigation to static analysis checker documentation through SARIF exports

FlameTop added a comment. I'm afraid we are seeing a build failure here on our local Windows checking MSVC build. Unfortunately I cannot find a public buildbot that uses the exact configuration that causes the assertion. The assertion we are seeing is Assertion failed: isa(Val) && "cast()

[clang-tools-extra] r349930 - [clang-tidy] Add export-fixes flag to clang-tidy-diff

Author: juliehockett Date: Fri Dec 21 09:25:27 2018 New Revision: 349930 URL: http://llvm.org/viewvc/llvm-project?rev=349930=rev Log: [clang-tidy] Add export-fixes flag to clang-tidy-diff Differential Revision: https://reviews.llvm.org/D55848 Modified:

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

This revision was automatically updated to reflect the committed changes. Closed by commit rL349930: [clang-tidy] Add export-fixes flag to clang-tidy-diff (authored by juliehockett, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

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

jordan_rose added a comment. I think Aaron and John have covered any comments I would make! I'm glad this came out so simply, though. I was afraid it'd be a much larger change than just expanding what was already there. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55865/new/

[PATCH] D56000: [compiler-rt] [xray] Disable alignas() for thread_local objects on NetBSD

krytarowski added inline comments. Comment at: lib/xray/xray_defs.h:22 +#if !SANITIZER_NETBSD +#define XRAY_TLS_ALIGNAS(x) alignas(x) I would switch the order, in order to remove unneeded negation. ``` #if SANITIZER_NETBSD ... #else ... #endif ``` `#define

[PATCH] D55869: Convert some ObjC retain/release msgSends to runtime calls

pete added a comment. Thanks for all the feedback so far. Is there anything else you'd like me to change before I can land this? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55869/new/ https://reviews.llvm.org/D55869 ___ cfe-commits

[PATCH] D56000: [compiler-rt] [xray] Disable alignas() for thread_local objects on NetBSD

mgorny marked 3 inline comments as done. mgorny added a comment. Updated as requested. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56000/new/ https://reviews.llvm.org/D56000 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D56000: [compiler-rt] [xray] Disable alignas() for thread_local objects on NetBSD

mgorny updated this revision to Diff 179297. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56000/new/ https://reviews.llvm.org/D56000 Files: lib/xray/xray_basic_logging.cc lib/xray/xray_defs.h lib/xray/xray_fdr_logging.cc Index: lib/xray/xray_fdr_logging.cc

r349924 - [AST][NFC] Pack CXXOperatorCallExpr

Author: brunoricci Date: Fri Dec 21 08:51:57 2018 New Revision: 349924 URL: http://llvm.org/viewvc/llvm-project?rev=349924=rev Log: [AST][NFC] Pack CXXOperatorCallExpr Use the space available in the bit-fields of Stmt. This saves 8 bytes per CXXOperatorCallExpr. NFC. Modified:

[PATCH] D56000: [compiler-rt] [xray] Disable alignas() for thread_local objects on NetBSD

krytarowski added inline comments. Comment at: lib/xray/xray_basic_logging.cc:58 -struct alignas(64) ThreadLocalData { +struct +/* TLD is not aligned properly on NetBSD, resulting in segfault */ krytarowski wrote: > mgorny wrote: > > krytarowski wrote: > > >

[PATCH] D56000: [compiler-rt] [xray] Disable alignas() for thread_local objects on NetBSD

krytarowski added inline comments. Comment at: lib/xray/xray_basic_logging.cc:58 -struct alignas(64) ThreadLocalData { +struct +/* TLD is not aligned properly on NetBSD, resulting in segfault */ mgorny wrote: > krytarowski wrote: > > Can we introduce a macro

[PATCH] D49754: Add -m(no-)spe, and e500 CPU definitions and support to clang

glaubitz added a comment. In D49754#1339161 , @vit9696 wrote: > Ok, I found the fix for the first crash that landed in 8.0 trunk. It works > fine for me if backported to 7.0.1: > https://reviews.llvm.org/D50461 That's awesome, thanks so much for

[PATCH] D56000: [compiler-rt] [xray] Disable alignas() for thread_local objects on NetBSD

mgorny marked an inline comment as done. mgorny added inline comments. Comment at: lib/xray/xray_basic_logging.cc:58 -struct alignas(64) ThreadLocalData { +struct +/* TLD is not aligned properly on NetBSD, resulting in segfault */ krytarowski wrote: > Can we

[PATCH] D49754: Add -m(no-)spe, and e500 CPU definitions and support to clang

vit9696 added a comment. Ok, I found the fix for the first crash that landed in 8.0 trunk. It works fine for me if backported to 7.0.1: https://reviews.llvm.org/D50461 Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D49754/new/ https://reviews.llvm.org/D49754

[PATCH] D56000: [compiler-rt] [xray] Disable alignas() for thread_local objects on NetBSD

krytarowski added inline comments. Comment at: lib/xray/xray_basic_logging.cc:58 -struct alignas(64) ThreadLocalData { +struct +/* TLD is not aligned properly on NetBSD, resulting in segfault */ Can we introduce a macro like: ``` #if SANITIZER_NETBSD #define

[PATCH] D56006: [AST] Fix a -Wimplicit-fallthrough warning in ScanfFormatString.cpp

riccibruno created this revision. riccibruno added a reviewer: hans. riccibruno added a project: clang. Herald added a subscriber: cfe-commits. This is showing up with some bots (eg

[PATCH] D55771: [AST] Store the callee and argument expressions of CallExpr in a trailing array.

This revision was automatically updated to reflect the committed changes. Closed by commit rL349910: [AST] Store the callee and argument expressions of CallExpr in a trailing array. (authored by brunoricci, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

r349910 - [AST] Store the callee and argument expressions of CallExpr in a trailing array.

Author: brunoricci Date: Fri Dec 21 07:20:32 2018 New Revision: 349910 URL: http://llvm.org/viewvc/llvm-project?rev=349910=rev Log: [AST] Store the callee and argument expressions of CallExpr in a trailing array. Since CallExpr::setNumArgs has been removed, it is now possible to store the callee

  1   2   >