r363919 - [clang] Fixing windows buildbot after D61552

Author: tyker Date: Thu Jun 20 03:27:14 2019 New Revision: 363919 URL: http://llvm.org/viewvc/llvm-project?rev=363919=rev Log: [clang] Fixing windows buildbot after D61552 Summary: original review : https://reviews.llvm.org/D61552 build bot faillure :

[PATCH] D62688: [Analyzer] Iterator Checkers - Model `empty()` method of containers

sanaanajjar231288 added a comment. In D62688#1549574 , @Szelethus wrote: > Hmm, an idea just popped into my head. I'm not sure whether we have a single > checker that does so much complicated (and totally awesome) modeling as > `IteratorChecker`. What

r363916 - [Testing] Dumping the graph requires assertions be enabled

Author: davezarzycki Date: Thu Jun 20 02:58:58 2019 New Revision: 363916 URL: http://llvm.org/viewvc/llvm-project?rev=363916=rev Log: [Testing] Dumping the graph requires assertions be enabled Modified: cfe/trunk/test/Analysis/exploded-graph-rewriter/escapes.c Modified:

r363920 - Revert "[clang] Fixing windows buildbot after D61552"

Author: tyker Date: Thu Jun 20 03:34:02 2019 New Revision: 363920 URL: http://llvm.org/viewvc/llvm-project?rev=363920=rev Log: Revert "[clang] Fixing windows buildbot after D61552" This reverts commit 5d5d2ca69e2b29b36db1a7dd1993ead7b7d2680f. has already been fixed by

r363917 - [clang][ASTMatchers] Add definition for cxxDeductionGuideDecl introduced in rL363855

Author: kadircet Date: Thu Jun 20 03:13:58 2019 New Revision: 363917 URL: http://llvm.org/viewvc/llvm-project?rev=363917=rev Log: [clang][ASTMatchers] Add definition for cxxDeductionGuideDecl introduced in rL363855 Modified: cfe/trunk/lib/ASTMatchers/ASTMatchersInternal.cpp Modified:

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

lildmh updated this revision to Diff 205785. lildmh marked 9 inline comments as done. lildmh added a comment. Address Alexey's comments CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59474/new/ https://reviews.llvm.org/D59474 Files: include/clang/AST/GlobalDecl.h

[PATCH] D28462: clang-format: Add new style option AlignConsecutiveMacros

phosek added a comment. Would it be possible to land this change? Do you need help with landing it? We've been waiting for this for a long time and we would like to start using it now that it has been accepted. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D28462/new/

[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers

lildmh added inline comments. Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:8739 +/// // For each component specified by this mapper: +/// if (currentComponent.hasMapper()) +/// (*currentComponent.Mapper())(rt_mapper_handle, arg_base, arg_begin,

[PATCH] D63600: [test][Driver] Fix Clang :: Driver/cl-response-file.c

ro created this revision. ro added a reviewer: rsmith. Herald added a subscriber: fedor.sergeev. Herald added a project: clang. `Clang :: Driver/cl-response-file.c` currently FAILs on Solaris: Command Output (stderr): -- /vol/llvm/src/clang/dist/test/Driver/cl-response-file.c:10:11: error:

[PATCH] D62373: [ASTImporter] Store import errors for Decls

martong updated this revision to Diff 205814. martong marked an inline comment as done. martong added a comment. - Remove unrelated change Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62373/new/ https://reviews.llvm.org/D62373 Files:

[PATCH] D62373: [ASTImporter] Store import errors for Decls

martong marked 3 inline comments as done. martong added inline comments. Comment at: clang/lib/AST/ASTImporter.cpp:5109 } else { // ODR violation. // FIXME HandleNameConflict + return make_error(ImportError::NameConflict); a_sidorin wrote: > Is

[PATCH] D60455: [SYCL] Implement SYCL device code outlining

Fznamznon updated this revision to Diff 205813. Fznamznon added a comment. Updated `sycl_kernel` attribute documentation. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60455/new/ https://reviews.llvm.org/D60455 Files:

r363932 - Add test cases for explicit casts when dumping the AST to JSON; NFC.

Author: aaronballman Date: Thu Jun 20 08:04:24 2019 New Revision: 363932 URL: http://llvm.org/viewvc/llvm-project?rev=363932=rev Log: Add test cases for explicit casts when dumping the AST to JSON; NFC. Added: cfe/trunk/test/AST/multistep-explicit-cast-json.c

[PATCH] D63578: AMDGPU: Add DS GWS sema builtins

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

r363935 - Removing a helper function that was trivial to inline into its only use; NFC.

Author: aaronballman Date: Thu Jun 20 08:10:45 2019 New Revision: 363935 URL: http://llvm.org/viewvc/llvm-project?rev=363935=rev Log: Removing a helper function that was trivial to inline into its only use; NFC. Modified: cfe/trunk/include/clang/AST/TextNodeDumper.h

[PATCH] D63579: [clang-scan-deps] print the dependencies to stdout and remove the need to use -MD options in the CDB

aganea added inline comments. Comment at: clang/tools/clang-scan-deps/ClangScanDeps.cpp:32 +class DependencyCollectorFactory { +public: Do you envision future uses for this factory? Comment at:

r363939 - AIX system headers need stdint.h and inttypes.h to be re-enterable

Author: xingxue Date: Thu Jun 20 08:36:32 2019 New Revision: 363939 URL: http://llvm.org/viewvc/llvm-project?rev=363939=rev Log: AIX system headers need stdint.h and inttypes.h to be re-enterable Summary: AIX system headers need stdint.h and inttypes.h to be re-enterable when macro _STD_TYPES_T

[PATCH] D59253: [AIX][libcxx] AIX system headers need stdint.h and inttypes.h to be re-enterable when macro _STD_TYPES_T is defined

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 rL363939: AIX system headers need stdint.h and inttypes.h to be re-enterable (authored by xingxue, committed by ). Changed

r363959 - [OpenMP] Add support for handling declare target to clause when unified memory is required

Author: gbercea Date: Thu Jun 20 11:04:47 2019 New Revision: 363959 URL: http://llvm.org/viewvc/llvm-project?rev=363959=rev Log: [OpenMP] Add support for handling declare target to clause when unified memory is required Summary: This patch adds support for the handling of the variables under

[PATCH] D63616: Implement `-fsanitize-coverage-whitelist` and `-fsanitize-coverage-blacklist` for clang

lebedev.ri added a comment. Test coverage missing. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63616/new/ https://reviews.llvm.org/D63616 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

r363980 - Rename CodeGenFunction::overlapFor* to getOverlapFor*.

Author: rsmith Date: Thu Jun 20 13:56:20 2019 New Revision: 363980 URL: http://llvm.org/viewvc/llvm-project?rev=363980=rev Log: Rename CodeGenFunction::overlapFor* to getOverlapFor*. Modified: cfe/trunk/lib/CodeGen/CGClass.cpp cfe/trunk/lib/CodeGen/CGDecl.cpp

[PATCH] D62738: [HIP] Support device_shadow variable

yaxunl marked an inline comment as done. yaxunl added a comment. In D62738#1538900 , @tra wrote: > So, the only thing this patch appears to do is make everything with this > attribute uninitialized on device side and give protected visibility. > If I

[PATCH] D63498: [RISC-V] Add -msave-restore and -mno-save-restore to clang driver

lenary updated this revision to Diff 205846. lenary added a comment. - Add CHECK lines for warnings Hopefully these will work regardless of race conditions in the merging of stdout and stderr. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

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

tmroeder marked an inline comment as done. tmroeder 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);

r363969 - [clang][NewPM] Move EntryExitInstrumenterPass to the start of the pipeline

Author: leonardchan Date: Thu Jun 20 12:35:25 2019 New Revision: 363969 URL: http://llvm.org/viewvc/llvm-project?rev=363969=rev Log: [clang][NewPM] Move EntryExitInstrumenterPass to the start of the pipeline This fixes CodeGen/x86_64-instrument-functions.c when running under the new pass

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

Sounds like a good start. On Thu, Jun 20, 2019 at 11:55 AM Tom Roeder via Phabricator wrote: > > tmroeder marked an inline comment as done. > tmroeder added inline comments. > > > > Comment at: clang-tools-extra/test/clang-tidy/linuxkernel-must-check-errs.c:6 > +// Prototypes of

[PATCH] D63613: [clang-tidy] Fail gracefully upon empty database fields

This revision was automatically updated to reflect the committed changes. Closed by commit rL363975: [clang-tidy] Fail gracefully upon empty database fields (authored by serge_sans_paille, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to

r363976 - P0840R2: support for [[no_unique_address]] attribute

Author: rsmith Date: Thu Jun 20 13:44:45 2019 New Revision: 363976 URL: http://llvm.org/viewvc/llvm-project?rev=363976=rev Log: P0840R2: support for [[no_unique_address]] attribute Summary: Add support for the C++2a [[no_unique_address]] attribute for targets using the Itanium C++ ABI. This

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

Anastasia added a comment. In D62574#1534159 , @ebevhan wrote: > >> 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. > >>

[PATCH] D63607: [DO NOT SUBMIT] [clang][driver] Prototype --driver-mode=fortran support for new flang

peterwaller-arm added a comment. Note: In case you see this early, the email isn't yet sent to the list. I'll link it here when it is, likely tomorrow. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63607/new/ https://reviews.llvm.org/D63607

r363952 - Store a pointer to the return value in a static alloca and let the debugger use that

Author: akhuang Date: Thu Jun 20 10:15:21 2019 New Revision: 363952 URL: http://llvm.org/viewvc/llvm-project?rev=363952=rev Log: Store a pointer to the return value in a static alloca and let the debugger use that as the variable address for NRVO variables. Subscribers: hiraditya, cfe-commits,

Re: [PATCH] D61879: WIP: Prototype of DSE optimizations for -ftrivial-auto-var-init

Not much, but it's not abandoned. I was interrupted by other stuff. Also I am concerned by the fact that zero initialization without any new optimization is cheaper than pattern even with patches in progress. On Thu, Jun 20, 2019 at 9:05 AM JF Bastien via Phabricator < revi...@reviews.llvm.org>

[PATCH] D63376: [clang] Small improvments after Adding APValue to ConstantExpr

rsmith added inline comments. Comment at: clang/include/clang/Sema/Sema.h:10298 + bool AllowFold = true, + bool StoreResult = true); ExprResult VerifyIntegerConstantExpression(Expr *E,

[PATCH] D60225: [clang-format] [PR19056] Add support for indenting class members and methods one level under the modifiers

3nids added a comment. Hi, may I ask why this is abandoned? We are eagerly waiting on this to move to clang-format. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60225/new/ https://reviews.llvm.org/D60225 ___

r363966 - Print additional information about @encode expressions when dumping the AST to JSON.

Author: aaronballman Date: Thu Jun 20 12:11:35 2019 New Revision: 363966 URL: http://llvm.org/viewvc/llvm-project?rev=363966=rev Log: Print additional information about @encode expressions when dumping the AST to JSON. Modified: cfe/trunk/include/clang/AST/JSONNodeDumper.h

r363972 - Fix crash and rejects-valid when a later template parameter or default

Author: rsmith Date: Thu Jun 20 12:49:13 2019 New Revision: 363972 URL: http://llvm.org/viewvc/llvm-project?rev=363972=rev Log: Fix crash and rejects-valid when a later template parameter or default template argument contains a backreference to a dependently-typed earlier parameter. In a case

[PATCH] D63617: [COFF, ARM64] Fix encoding of __debugbreak

rnk added a comment. I think it would be preferable to make `llvm.debugtrap` emit `brk #0xF000` on aarch64-windows-*, so other frontends (Rust etc) get the right behavior by default. Right now, AArch64 doesn't do anything special for DEBUGTRAP, so we get the default behavior of `llvm.trap`.

[PATCH] D63451: P0840R2: support for [[no_unique_address]] attribute

rjmccall accepted this revision. rjmccall added inline comments. Comment at: include/clang/Basic/Attr.td:326 + // const llvm::Triple = Target.getTriple(); + code CustomCode = [{}]; } Thanks! Comment at: lib/CodeGen/CGExprAgg.cpp:1850

[PATCH] D63451: P0840R2: support for [[no_unique_address]] attribute

rsmith added a comment. In D63451#1549563 , @rjmccall wrote: > Can this attribute not be applied to a base class, or to a type? The standard attribute forbids that right now. We could add a custom attribute that permits it, but we're required to

[PATCH] D63161: Devirtualize destructor of final class.

rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. Thanks! Minor comment then feel free to commit. Comment at: lib/CodeGen/CGExprCXX.cpp:1877 + DevirtualizedDtor->getParent(); + if (getCXXRecord(Base)

[PATCH] D63161: Devirtualize destructor of final class.

rsmith added inline comments. Comment at: test/CodeGenCXX/devirtualize-dtor-final.cpp:20 + void evil(D *p) { +// CHECK-NOT: call void@_ZN5Test11DD1Ev +delete p; Missing space after `void` here Repository: rC Clang CHANGES SINCE LAST ACTION

[PATCH] D63623: [clang-tidy] Add a check on expected return values of posix functions (except posix_openpt) in Android module.

jcai19 added a comment. In D63623#1552679 , @lebedev.ri wrote: > Why is this in android module? > Is that android-specific behavior, or posix? I implemented it for Andriod as requested, but it would be completely fine to move it if it is better to

[PATCH] D63623: [clang-tidy] Add a check on expected return values of posix functions (except posix_openpt) in Android module.

george.burgess.iv added a comment. just a few drive-by nits/comments from me. as usual, not super familiar with clang-tidy, so i won't be able to stamp this. thanks! Comment at: clang-tools-extra/clang-tidy/android/PosixReturnCheck.cpp:23 + binaryOperator( +

[PATCH] D63600: [test][Driver] Fix Clang :: Driver/cl-response-file.c

jkorous accepted this revision. jkorous added a comment. This revision is now accepted and ready to land. Herald added a subscriber: dexonsmith. LGTM Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63600/new/ https://reviews.llvm.org/D63600

[PATCH] D60974: Clang IFSO driver action.

plotfi added a comment. This was re-landed in r363948 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60974/new/ https://reviews.llvm.org/D60974 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

r363961 - [X86] Make _mm_mask_cvtps_ph, _mm_maskz_cvtps_ph, _mm256_mask_cvtps_ph, and _mm256_maskz_cvtps_ph aliases for their corresponding cvt_roundps_ph intrinsic.

Author: ctopper Date: Thu Jun 20 11:24:29 2019 New Revision: 363961 URL: http://llvm.org/viewvc/llvm-project?rev=363961=rev Log: [X86] Make _mm_mask_cvtps_ph, _mm_maskz_cvtps_ph, _mm256_mask_cvtps_ph, and _mm256_maskz_cvtps_ph aliases for their corresponding cvt_roundps_ph intrinsic. These

[PATCH] D63616: Implement `-fsanitize-coverage-whitelist` and `-fsanitize-coverage-blacklist` for clang

tuktuk created this revision. tuktuk added reviewers: kcc, morehouse. tuktuk added projects: clang, Sanitizers. Herald added subscribers: llvm-commits, cfe-commits, hiraditya. Herald added a project: LLVM. This commit adds two command-line options to clang. These options let the user decide which

r363971 - [clang][NewPM] Do not eliminate available_externally durng `-O2 -flto` runs

Author: leonardchan Date: Thu Jun 20 12:44:51 2019 New Revision: 363971 URL: http://llvm.org/viewvc/llvm-project?rev=363971=rev Log: [clang][NewPM] Do not eliminate available_externally durng `-O2 -flto` runs This fixes CodeGen/available-externally-suppress.c when the new pass manager is turned

[PATCH] D63579: [clang-scan-deps] print the dependencies to stdout and remove the need to use -MD options in the CDB

aganea added inline comments. Comment at: clang/tools/clang-scan-deps/ClangScanDeps.cpp:32 +class DependencyCollectorFactory { +public: arphaman wrote: > aganea wrote: > > Do you envision future uses for this factory? > Most likely, yes. > > I don't want to

[PATCH] D63623: [clang-tidy] Add a check on expected return values of posix functions (except posix_openpt) in Android module.

lebedev.ri added a comment. Why is this in android module? Is that android-specific behavior, or posix? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63623/new/ https://reviews.llvm.org/D63623 ___

[PATCH] D63613: [clang-tidy] Fail gracefully upon empty database fields

jkorous accepted this revision. jkorous added a comment. This revision is now accepted and ready to land. Herald added a subscriber: dexonsmith. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63613/new/ https://reviews.llvm.org/D63613

[PATCH] D63580: [clang][NewPM] Do not eliminate available_externally durng `-O2 -flto` runs

This revision was automatically updated to reflect the committed changes. leonardchan marked an inline comment as done. Closed by commit rG97dc622ab3f7: [clang][NewPM] Do not eliminate available_externally durng `-O2 -flto` runs (authored by leonardchan). Changed prior to commit:

[PATCH] D63579: [clang-scan-deps] print the dependencies to stdout and remove the need to use -MD options in the CDB

arphaman added inline comments. Comment at: clang/tools/clang-scan-deps/ClangScanDeps.cpp:32 +class DependencyCollectorFactory { +public: aganea wrote: > Do you envision future uses for this factory? Most likely, yes. I don't want to lock-in into creating the

[PATCH] D63579: [clang-scan-deps] print the dependencies to stdout and remove the need to use -MD options in the CDB

arphaman updated this revision to Diff 205886. arphaman marked 3 inline comments as done. arphaman added a comment. removed the lock CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63579/new/ https://reviews.llvm.org/D63579 Files: clang/include/clang/Frontend/Utils.h

[PATCH] D63613: [clang-tidy] Fail gracefully upon empty database fields

serge-sans-paille added a comment. Thanks @jkorous for the fast review o/ Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63613/new/ https://reviews.llvm.org/D63613 ___ cfe-commits mailing list

[PATCH] D63451: P0840R2: support for [[no_unique_address]] attribute

This revision was automatically updated to reflect the committed changes. Closed by commit rL363976: P0840R2: support for [[no_unique_address]] attribute (authored by rsmith, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Repository: rL LLVM CHANGES

[PATCH] D62156: [Sema][PR41730] Diagnose addr space mismatch while constructing objects

Anastasia marked 2 inline comments as done. Anastasia added inline comments. Comment at: test/SemaCXX/address-space-ctor.cpp:11 +//expected-note@-6{{candidate constructor (the implicit move constructor) not viable: no known conversion from 'int' to 'MyType &&' for 1st

[PATCH] D63361: Pretend NRVO variables are references so they can be found by debug info

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 rL363952: Store a pointer to the return value in a static alloca and let the debugger use… (authored by akhuang, committed

r363963 - [NFC] Fix for InterfaceStubs tests (adding REQUIRES: x86-registered-target).

Author: zer0 Date: Thu Jun 20 11:28:21 2019 New Revision: 363963 URL: http://llvm.org/viewvc/llvm-project?rev=363963=rev Log: [NFC] Fix for InterfaceStubs tests (adding REQUIRES: x86-registered-target). clang-hexagon-elf bot was failing with: 'No available targets are compatible with triple

[PATCH] D63621: [git-clang-format] recognize hxx as a C++ file

vmiklos created this revision. vmiklos added reviewers: jbcoe, djasper. Herald added subscribers: cfe-commits, kadircet, ilya-biryukov. Herald added a project: clang. clangd, clang-tidy, etc does that already, no reason why git-clang-format should skip hxx files. Repository: rC Clang

r363975 - [clang-tidy] Fail gracefully upon empty database fields

Author: serge_sans_paille Date: Thu Jun 20 13:25:59 2019 New Revision: 363975 URL: http://llvm.org/viewvc/llvm-project?rev=363975=rev Log: [clang-tidy] Fail gracefully upon empty database fields Fix bz#42281 Differential Revision: https://reviews.llvm.org/D63613 Modified:

[clang-tools-extra] r363975 - [clang-tidy] Fail gracefully upon empty database fields

Author: serge_sans_paille Date: Thu Jun 20 13:25:59 2019 New Revision: 363975 URL: http://llvm.org/viewvc/llvm-project?rev=363975=rev Log: [clang-tidy] Fail gracefully upon empty database fields Fix bz#42281 Differential Revision: https://reviews.llvm.org/D63613 Added:

[PATCH] D63603: [ASTImporter] Propagate error from ImportDeclContext

jkorous added a comment. I don't have the insight to LGTM the whole change - just a nit about implementation detail. Comment at: clang/lib/AST/ASTImporter.cpp:1724 + }; + DefinitionCompleter CompleterRAII(To); You might consider using just a lambda with

[PATCH] D63108: [OpenMP] Add support for handling declare target to clause when unified memory is required

This revision was automatically updated to reflect the committed changes. Closed by commit rL363959: [OpenMP] Add support for handling declare target to clause when unified memory… (authored by gbercea, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits.

[PATCH] D63617: [COFF, ARM64] Fix encoding of __debugbreak

rnk added a comment. Even if `BRK #0xF000` is a Windows convention, it's still possible for ISel to select different instructions for different OSs, and I'd prefer to implement it that way. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63617/new/

[PATCH] D63451: P0840R2: support for [[no_unique_address]] attribute

rjmccall added a comment. In D63451#1552609 , @rsmith wrote: > In D63451#1549563 , @rjmccall wrote: > > > Can this attribute not be applied to a base class, or to a type? > > > The standard attribute forbids that

[PATCH] D63613: [clang-tidy] Fail gracefully upon empty database fields

serge-sans-paille created this revision. serge-sans-paille added a reviewer: alexfh. Herald added subscribers: cfe-commits, xazax.hun. Herald added a project: clang. fix https://bugs.llvm.org/show_bug.cgi?id=42281 Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D63613 Files:

[PATCH] D63617: [COFF, ARM64] Fix encoding of __debugbreak

TomTan created this revision. TomTan added reviewers: efriedma, rnk. Herald added subscribers: cfe-commits, kristof.beyls, javed.absar. Herald added a project: clang. On Windows ARM64, intrinsic `__debugbreak` should be compiled into `brk #0xF000` which is different from llvm intrinsic

r363973 - Fix passing structs and AVX vectors through sysv_abi

Author: rnk Date: Thu Jun 20 13:07:20 2019 New Revision: 363973 URL: http://llvm.org/viewvc/llvm-project?rev=363973=rev Log: Fix passing structs and AVX vectors through sysv_abi Do this the same way we did it for ms_abi in r324594. Fixes PR36806. Added: cfe/trunk/test/CodeGen/sysv_abi.c

[PATCH] D63451: P0840R2: support for [[no_unique_address]] attribute

rsmith added inline comments. Comment at: lib/CodeGen/CGExprAgg.cpp:1850 +AggValueSlot::Overlap_t +CodeGenFunction::overlapForFieldInit(const FieldDecl *FD) { + if (!FD->hasAttr() || !FD->getType()->isRecordType()) rsmith wrote: > rjmccall wrote: > >

[PATCH] D63451: P0840R2: support for [[no_unique_address]] attribute

rsmith updated this revision to Diff 205892. rsmith marked 4 inline comments as done. rsmith added a comment. - Use custom code to specify CXXABI requirements on attributes. - Remove dead code that would have handled [[no_unique_address]] in C. - Extend documentation to include an example and to

[PATCH] D63600: [test][Driver] Fix Clang :: Driver/cl-response-file.c

This revision was automatically updated to reflect the committed changes. Closed by commit rL363985: [test][Driver] Fix Clang :: Driver/cl-response-file.c (authored by ro, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit:

r363985 - [test][Driver] Fix Clang :: Driver/cl-response-file.c

Author: ro Date: Thu Jun 20 14:33:09 2019 New Revision: 363985 URL: http://llvm.org/viewvc/llvm-project?rev=363985=rev Log: [test][Driver] Fix Clang :: Driver/cl-response-file.c Clang :: Driver/cl-response-file.c currently FAILs on Solaris: Command Output (stderr): --

[PATCH] D63625: [CodeGen][test] Use -fno-discard-value-names for better test support

rupprecht created this revision. rupprecht added reviewers: rnk, akhuang, aprantl. Herald added a project: clang. Herald added a subscriber: cfe-commits. Depending on how clang is built, it may discard the IR names and use names like `%2` instead of `%result.ptr`, causing tests that rely on the

[PATCH] D62738: [HIP] Support device_shadow variable

yaxunl added a comment. > The problem is that we do not see generic usage of > Although there is no texture specific handling on the compiler side, there > is texture specific handling of symbols Please ignore this comment. It is some old comment submitted by accident. CHANGES SINCE LAST

r363948 - [clang-ifs] Clang Interface Stubs, first version (second landing attempt).

Author: zer0 Date: Thu Jun 20 09:59:48 2019 New Revision: 363948 URL: http://llvm.org/viewvc/llvm-project?rev=363948=rev Log: [clang-ifs] Clang Interface Stubs, first version (second landing attempt). This change reverts r363649; effectively re-landing r363626. At this point

[PATCH] D63577: [clang][NewPM] Move EntryExitInstrumenterPass to the start of the pipeline

This revision was automatically updated to reflect the committed changes. Closed by commit rL363969: [clang][NewPM] Move EntryExitInstrumenterPass to the start of the pipeline (authored by leonardchan, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed

[PATCH] D63617: [COFF, ARM64] Fix encoding of __debugbreak

TomTan added a comment. In D63617#1552561 , @rnk wrote: > I think it would be preferable to make `llvm.debugtrap` emit `brk #0xF000` on > aarch64-windows-*, so other frontends (Rust etc) get the right behavior by > default. Right now, AArch64 doesn't do

[PATCH] D63451: P0840R2: support for [[no_unique_address]] attribute

rsmith marked an inline comment as done. rsmith added inline comments. Comment at: lib/CodeGen/CGExprAgg.cpp:1850 +AggValueSlot::Overlap_t +CodeGenFunction::overlapForFieldInit(const FieldDecl *FD) { + if (!FD->hasAttr() || !FD->getType()->isRecordType())

[PATCH] D63623: Add a check on expected return values of posix functions (except posix_openpt) in Android module.

jcai19 created this revision. Herald added subscribers: cfe-commits, mgorny, srhines. Herald added a project: clang. Checks if any calls to posix functions (except posix_openpt) expect negative return values. These functions return either 0 on success or an errno on failure, which is positive

r363986 - AMDGPU: Add DS GWS sema builtins

Author: arsenm Date: Thu Jun 20 14:33:57 2019 New Revision: 363986 URL: http://llvm.org/viewvc/llvm-project?rev=363986=rev Log: AMDGPU: Add DS GWS sema builtins Modified: cfe/trunk/include/clang/Basic/BuiltinsAMDGPU.def cfe/trunk/test/CodeGenOpenCL/builtins-amdgcn-ci.cl

[PATCH] D63625: [CodeGen][test] Use -fno-discard-value-names for better test support

lebedev.ri added a comment. In D63625#1552756 , @akhuang wrote: > Looks good, alternatively I think we can just change `%result.ptr` into a > variable match? Why isn't this using FileCheck variable matching in the first place? That's the least fragile

[PATCH] D63625: [CodeGen][test] Use -fno-discard-value-names for better test support

akhuang added a comment. Looks good, alternatively I think we can just change `%result.ptr` into a variable match? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63625/new/ https://reviews.llvm.org/D63625

[PATCH] D63538: [analyzer][CFG] Return the correct terminator condition

Szelethus updated this revision to Diff 205909. Szelethus added a comment. Addressing reviewer feedback! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63538/new/ https://reviews.llvm.org/D63538 Files: clang/include/clang/Analysis/CFG.h clang/lib/Analysis/CFG.cpp Index:

[PATCH] D62611: [analyzer][Dominators] Add unittests

Szelethus added a comment. Gentle ping :^) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62611/new/ https://reviews.llvm.org/D62611 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D62619: [analyzer][IDF] Add a control dependency calculator + a new debug checker

Szelethus updated this revision to Diff 205917. Szelethus added a comment. Rebase on previous patches. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62619/new/ https://reviews.llvm.org/D62619 Files: clang/include/clang/Analysis/Analyses/Dominators.h

[PATCH] D63623: [clang-tidy] Add a check on expected return values of posix functions (except posix_openpt) in Android module.

Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/clang-tidy/android/PosixReturnCheck.h:1 +//===--- PosixReturnCheck.h - clang-tidy*- C++ -*-// +// Please use - before *- C++ -* to fill line. See other checks as

[PATCH] D62883: [analyzer] Track conditions of terminator statements on which the reported node depends on

NoQ added a comment. Oh, wait, no, loops. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62883/new/ https://reviews.llvm.org/D62883 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D62883: [analyzer] Track conditions of terminator statements on which the reported node depends on

NoQ added a comment. In D62883#1552969 , @xazax.hun wrote: > In D62883#1552870 , @Szelethus wrote: > > > - Uniqueing already tracked conditions as an (`Expr`, `ExplodedNode`) pair > > instead of on `Expr` > > > I

[PATCH] D63538: [analyzer][CFG] Return the correct terminator condition

NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Ok, thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63538/new/ https://reviews.llvm.org/D63538 ___ cfe-commits mailing list

r363994 - [X86] Change LL to O in the definitions for the vp2intersect builtins.

Author: ctopper Date: Thu Jun 20 15:19:16 2019 New Revision: 363994 URL: http://llvm.org/viewvc/llvm-project?rev=363994=rev Log: [X86] Change LL to O in the definitions for the vp2intersect builtins. This is needed to support OpenCL where long long is 128 bits. This was done for the other

[PATCH] D63625: [CodeGen][test] Use -fno-discard-value-names for better test support

This revision was automatically updated to reflect the committed changes. Closed by commit rG73986707bd57: [CodeGen][test] Use FileCheck variable matchers for better test support (authored by rupprecht). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D62883: [analyzer] Track conditions of terminator statements on which the reported node depends on

xazax.hun added a comment. In D62883#1552870 , @Szelethus wrote: > - Now using `CFGBlock::getTerminatorConditionExpr()` > - Uniqueing already tracked conditions as an (`Expr`, `ExplodedNode`) pair > instead of on `Expr` I would be surprised to see the

[PATCH] D63636: [PowerPC][Altivec] Fix offsets for vec_xl and vec_xst

nemanjai created this revision. nemanjai added reviewers: hfinkel, jsji, rzurob, saghir. Herald added a subscriber: kbarton. Herald added a project: clang. As we currently have it implemented in altivec.h, the offsets for these two intrinsics are element offsets. The documentation in the ABI (as

[PATCH] D63156: [clang][NewPM] Add -fno-experimental-new-pass-manager to tests

chandlerc accepted this revision. chandlerc added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63156/new/ https://reviews.llvm.org/D63156

[PATCH] D63156: [clang][NewPM] Add -fno-experimental-new-pass-manager to tests

leonardchan updated this revision to Diff 205906. leonardchan marked 3 inline comments as done. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63156/new/ https://reviews.llvm.org/D63156 Files: clang/test/CodeGen/aggregate-assign-call.c

[PATCH] D63579: [clang-scan-deps] print the dependencies to stdout and remove the need to use -MD options in the CDB

arphaman marked an inline comment as done. arphaman added inline comments. Comment at: clang/tools/clang-scan-deps/ClangScanDeps.cpp:32 +class DependencyCollectorFactory { +public: aganea wrote: > arphaman wrote: > > aganea wrote: > > > Do you envision future

[PATCH] D62368: Add support for Hygon Dhyana processor

fanjinke added reviewers: 01alchemist, 4tXJ7f. fanjinke added a comment. Hi, Please help with the patch reveiw. Hi cryptoad, @cryptoad IS there anything incorrectly? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62368/new/

[PATCH] D63174: [clang][NewPM] Add RUNS for tests that produce slightly different IR under new PM

craig.topper added a comment. I'm going to try to work on the X86 tests. Can we hold off on committing those? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63174/new/ https://reviews.llvm.org/D63174

r363995 - [analyzer] DeadStores: Update the crude suppression for files generated by IIG.

Author: dergachev Date: Thu Jun 20 15:29:40 2019 New Revision: 363995 URL: http://llvm.org/viewvc/llvm-project?rev=363995=rev Log: [analyzer] DeadStores: Update the crude suppression for files generated by IIG. They changed the comments that we were looking for. Modified:

  1   2   >