[PATCH] D58106: [compiler-rt] [profile] Provide lprofGetHostName for all windows environments

mstorsjo created this revision. mstorsjo added reviewers: rnk, compnerd. Herald added subscribers: Sanitizers, llvm-commits, jdoerfert, dberris. Herald added projects: LLVM, Sanitizers. This function doesn't use anything MSVC specific but works fine for any `_WIN32` target. Repository: rCRT C

Re: r353495 - Variable auto-init: fix __block initialization

Merged to release_80 in r353807. On Fri, Feb 8, 2019 at 2:28 AM JF Bastien via cfe-commits wrote: > > Author: jfb > Date: Thu Feb 7 17:29:17 2019 > New Revision: 353495 > > URL: http://llvm.org/viewvc/llvm-project?rev=353495&view=rev > Log: > Variable auto-init: fix __block initialization > > Su

[PATCH] D57984: PR40642: Fix determination of whether the final statement of a statementexpression is a discarded-value expression.

ABataev added a comment. Herald added a subscriber: jdoerfert. In D57984#1394167 , @rsmith wrote: > In D57984#1394067 , @ABataev wrote: > > > In D57984#1394050 , @rsmith wrot

[PATCH] D57427: [CodeGen][ObjC] Fix assert on calling `__builtin_constant_p` with ObjC objects.

hans added a comment. Herald added a subscriber: jdoerfert. > Also looks like in practice it shouldn't be a serious problem, > `__builtin_constant_p` can be optimized out. For example, see > https://godbolt.org/z/e1b4dB Okay, I'm not going to worry about it then. Thanks. Repository: rL LLVM

[PATCH] D41005: Reuse preamble even if an unsaved file does not exist

nik updated this revision to Diff 186429. nik marked an inline comment as done. nik added a comment. Herald added a subscriber: jdoerfert. Herald added a project: clang. Addressed comment. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D41005/new/ https://reviews.l

[clang-tools-extra] r353821 - [clangd] Fix use-after-free in XRefs

Author: sammccall Date: Tue Feb 12 02:38:45 2019 New Revision: 353821 URL: http://llvm.org/viewvc/llvm-project?rev=353821&view=rev Log: [clangd] Fix use-after-free in XRefs Modified: clang-tools-extra/trunk/clangd/XRefs.cpp Modified: clang-tools-extra/trunk/clangd/XRefs.cpp URL: http://llvm

[PATCH] D58111: [Sema] Fix a crash in access checking for deduction guides

ilya-biryukov created this revision. ilya-biryukov added a reviewer: sammccall. Herald added a project: clang. See the added test for a repro. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D58111 Files: clang/lib/Sema/SemaDeclCXX.cpp clang/test/Sema/crash-deduction-guide-a

[PATCH] D41005: Reuse preamble even if an unsaved file does not exist

nik added a comment. Meh, something changed in the meanwhile. ReparseReusesPreambleAfterUnsavedFileWasRemovedFromDisk fails now. Looking into it. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D41005/new/ https://reviews.llvm.org/D41005 ___

Re: r353411 - Fix r350643 to limit COFF emission to <= 32 BYTES instead of BITS.

Merged to 8.0 in r353825. On Thu, Feb 7, 2019 at 4:13 PM Erich Keane via cfe-commits wrote: > > Author: erichkeane > Date: Thu Feb 7 07:14:11 2019 > New Revision: 353411 > > URL: http://llvm.org/viewvc/llvm-project?rev=353411&view=rev > Log: > Fix r350643 to limit COFF emission to <= 32 BYTES in

[PATCH] D58111: [Sema] Fix a crash in access checking for deduction guides

sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang/lib/Sema/SemaDeclCXX.cpp:3178 auto *TD = DG->getDeducedTemplate(); - if (AS != TD->getAccess()) { + bool InSameScope = TD->getDec

Re: r353431 - [OpenCL][PR40603] In C++ preserve compatibility with OpenCL C v2.0

Merged to 8.0 in r353826. On Thu, Feb 7, 2019 at 6:32 PM Anastasia Stulova via cfe-commits wrote: > > Author: stulova > Date: Thu Feb 7 09:32:37 2019 > New Revision: 353431 > > URL: http://llvm.org/viewvc/llvm-project?rev=353431&view=rev > Log: > [OpenCL][PR40603] In C++ preserve compatibility w

Re: r353493 - [COFF, ARM64] Fix types for _ReadStatusReg, _WriteStatusReg

Merged to 8.0 in r353828. On Fri, Feb 8, 2019 at 2:17 AM Eli Friedman via cfe-commits wrote: > > Author: efriedma > Date: Thu Feb 7 17:17:49 2019 > New Revision: 353493 > > URL: http://llvm.org/viewvc/llvm-project?rev=353493&view=rev > Log: > [COFF, ARM64] Fix types for _ReadStatusReg, _WriteSta

Re: r353402 - [clang-cl] support /Oy- on aarch64

Merged to 8.0 in r353829. On Thu, Feb 7, 2019 at 1:46 PM Martin Storsjo via cfe-commits wrote: > > Author: mstorsjo > Date: Thu Feb 7 04:46:49 2019 > New Revision: 353402 > > URL: http://llvm.org/viewvc/llvm-project?rev=353402&view=rev > Log: > [clang-cl] support /Oy- on aarch64 > > MSVC support

Re: r353656 - long double is double on OpenBSD/NetBSD/PPC.

Merged to 8.0 in r353831. On Mon, Feb 11, 2019 at 3:52 AM Brad Smith via cfe-commits wrote: > > Author: brad > Date: Sun Feb 10 18:53:16 2019 > New Revision: 353656 > > URL: http://llvm.org/viewvc/llvm-project?rev=353656&view=rev > Log: > long double is double on OpenBSD/NetBSD/PPC. > > Patch by

[PATCH] D41005: Reuse preamble even if an unsaved file does not exist

nik updated this revision to Diff 186436. nik added a comment. > Meh, something changed in the meanwhile. > ReparseReusesPreambleAfterUnsavedFileWasRemovedFromDisk fails now. Looking > into it. No, it's just me ;) I've referenced the header file wrong. Repository: rC Clang CHANGES SINCE LA

r353836 - Disable test after r353718, r353725, r353729 while I investigate

Author: nico Date: Tue Feb 12 04:40:56 2019 New Revision: 353836 URL: http://llvm.org/viewvc/llvm-project?rev=353836&view=rev Log: Disable test after r353718, r353725, r353729 while I investigate Modified: cfe/trunk/test/CodeGen/ms-x86-intrinsics.c Modified: cfe/trunk/test/CodeGen/ms-x86-int

Re: r353729 - Attempt to pacify bots more after r353718 and r353725

Thanks for reporting that this depends on the host compiler. I disabled the test again in r353836 and will look into why the output is different depending on if host cc is gcc or clang. On Tue, Feb 12, 2019 at 2:40 AM Mikael Holmén wrote: > Same thing for me with our downstream build bots. > >

[PATCH] D54978: Move the SMT API to LLVM

thakis added a comment. Herald added a subscriber: jdoerfert. >> If so, I don't understand why the default setting is important to you and >> why this doesn't work for you. (I don't disagree with the default being off, >> I'm just confused why things don't work for you.) > > As I have stated se

[PATCH] D17444: PR26672: [MSVC] Clang does not recognize "static_assert" keyword in C mode

thakis added a comment. Herald added a subscriber: jdoerfert. I don't have an opinion if this patch here or the assert wrapper is better, but I feel somewhat strongly that we should land one or the other so that the standard C program #include static_assert(4 == 4 , ""); builds in clan

r353837 - Fixing a typo; NFC.

Author: aaronballman Date: Tue Feb 12 05:04:11 2019 New Revision: 353837 URL: http://llvm.org/viewvc/llvm-project?rev=353837&view=rev Log: Fixing a typo; NFC. Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td cfe/trunk/lib/Sema/SemaDeclAttr.cpp Modified: cfe/trunk/include/cl

r353838 - Renaming this diagnostic to not conflict with another; NFC.

Author: aaronballman Date: Tue Feb 12 05:13:35 2019 New Revision: 353838 URL: http://llvm.org/viewvc/llvm-project?rev=353838&view=rev Log: Renaming this diagnostic to not conflict with another; NFC. Amends r353837 which renamed the diagnostics to conflict. Modified: cfe/trunk/include/clang/B

r353839 - Renaming yet another diagnostic to not conflict; NFC.

Author: aaronballman Date: Tue Feb 12 05:19:02 2019 New Revision: 353839 URL: http://llvm.org/viewvc/llvm-project?rev=353839&view=rev Log: Renaming yet another diagnostic to not conflict; NFC. Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td cfe/trunk/lib/Sema/SemaDeclAttr.c

[PATCH] D57984: PR40642: Fix determination of whether the final statement of a statementexpression is a discarded-value expression.

aaron.ballman added a comment. Considering that this has been fertile ground for buggy edge cases, should we revert my original changes from the 8.0 branch to give this more time to bake on trunk before releasing? Or do you think this is fine to move onto the release branch once finalized? =

[PATCH] D58095: [clang-tidy] Make google-objc-function-naming ignore implicit functions 🙈

aaron.ballman added inline comments. Herald added a subscriber: jdoerfert. Comment at: clang-tools-extra/test/clang-tidy/google-objc-function-naming.m:10 +// function would be declared in a system header. +int printf(const char *, ...); // NOLINT(google-objc-function-naming) + -

[PATCH] D57236: [ASTImporter] Unify redecl chain tests as type parameterized tests

martong updated this revision to Diff 186445. martong added a comment. Herald added a subscriber: jdoerfert. - Remove numbers when possible - Use disabled tests instead of commented out tests Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57236/new/ https://review

[PATCH] D57236: [ASTImporter] Unify redecl chain tests as type parameterized tests

martong marked 4 inline comments as done. martong added a comment. Thanks for the review Alexei! Comment at: unittests/AST/ASTImporterTest.cpp:3549 + void TypedTest_ImportDefinitionThenPrototype() { +Decl *FromTU0 = getTuDecl(getDefinition(), Lang_CXX, "input0.cc"); +D

Re: r353729 - Attempt to pacify bots more after r353718 and r353725

On 2/12/19 1:41 PM, Nico Weber wrote: > Thanks for reporting that this depends on the host compiler. > > I disabled the test again in r353836 and will look into why the output > is different depending on if host cc is gcc or clang. > Good. Thanks! When I've seen things like this before it's

[PATCH] D58111: [Sema] Fix a crash in access checking for deduction guides

ilya-biryukov updated this revision to Diff 186447. ilya-biryukov marked an inline comment as done. ilya-biryukov added a comment. - Address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58111/new/ https://reviews.llvm.org/D58111 Files:

[PATCH] D57948: [Sema] Fix a regression introduced in "[AST][Sema] Remove CallExpr::setNumArgs"

riccibruno updated this revision to Diff 186449. riccibruno added a comment. Rewrote the patch to make it local to `BuildResolvedCallExpr`. It is still a hack though. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57948/new/ https://reviews.llvm.org/D57948 Files

[PATCH] D58111: [Sema] Fix a crash in access checking for deduction guides

This revision was automatically updated to reflect the committed changes. Closed by commit rC353840: [Sema] Fix a crash in access checking for deduction guides (authored by ibiryukov, committed by ). Changed prior to commit: https://reviews.llvm.org/D58111?vs=186447&id=186450#toc Repository:

r353840 - [Sema] Fix a crash in access checking for deduction guides

Author: ibiryukov Date: Tue Feb 12 06:21:44 2019 New Revision: 353840 URL: http://llvm.org/viewvc/llvm-project?rev=353840&view=rev Log: [Sema] Fix a crash in access checking for deduction guides Summary: See the added test for a repro. Reviewers: sammccall Reviewed By: sammccall Subscribers: c

[PATCH] D58074: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier

ABataev added a comment. Herald added a subscriber: jdoerfert. Also, need some kind of the stubbed codegen for the mapper Comment at: include/clang/AST/OpenMPClause.h:4236 + static OMPMapClause * + Create(const ASTContext &C, SourceLocation StartLoc, SourceLocation LParenLoc

[PATCH] D45978: dllexport const variables must have external linkage.

zahiraam added a subscriber: z. zahiraam added inline comments. Comment at: test/Sema/dllexport-1.c:8 + +// CHECK: @y = common dso_local dllexport global i32 0, align 4 + aaron.ballman wrote: > Are x and z also exported as expected? Only x and y are exported. *

[PATCH] D45978: dllexport const variables must have external linkage.

zahiraam updated this revision to Diff 186460. zahiraam marked 3 inline comments as done. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D45978/new/ https://reviews.llvm.org/D45978 Files: lib/Sema/SemaDecl.cpp test/CodeGen/dllexport.c test/Sema/dllexport-1.cpp test/Sema/dllexport-2

[PATCH] D58120: [Builtins] Treat `bcmp` as a builtin.

courbet created this revision. courbet added a reviewer: jyknight. Herald added a subscriber: kristina. Herald added a project: clang. This makes it consistent with `memcmp` and `__builtin_bcmp`. Also see the discussion in https://reviews.llvm.org/D56593. Repository: rC Clang https://reviews

[PATCH] D58121: [analyzer][WIP] Attempt to fix traversing bindings of non-base regions in ClusterAnalysis

xazax.hun created this revision. xazax.hun added a reviewer: NoQ. Herald added subscribers: gamesh411, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware, whisperity. Herald added a project: clang. Follow up for a problem described in https://reviews.l

[PATCH] D57230: [analyzer] Toning down invalidation a bit

xazax.hun added a comment. Herald added a subscriber: jdoerfert. Experimental patch is up in https://reviews.llvm.org/D58121 Unfortunately, it is not perfect yet. Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57230/new/ https://reviews.llvm.org/D57230 _

[PATCH] D57898: CodeGen: Fix PR40605: split constant structures generated by -ftrivial-auto-var-init when emitting initializators

glider marked 4 inline comments as done. glider added inline comments. Herald added a subscriber: jdoerfert. Comment at: tools/clang/test/CodeGenCXX/auto-var-init.cpp:496 // ZERO-LABEL: @test_smallpartinit_uninit() // ZERO: call void @llvm.memset{{.*}}, i8 0,

[PATCH] D58122: Restore Check for Unreachable Exit Block in -Winfinite-recursion

CodaFi created this revision. CodaFi added reviewers: steven_wu, rtrieu. Herald added a project: clang. Herald added a subscriber: cfe-commits. When this was rewritten in D43737 , the logic changed to better explore infinite loops. The check for a reachable exit b

[PATCH] D58067: [Analyzer] Crash fix for FindLastStoreBRVisitor

baloghadamsoftware updated this revision to Diff 186470. baloghadamsoftware marked 3 inline comments as not done. baloghadamsoftware added a comment. Herald added a subscriber: jdoerfert. Function renamed, comment updated. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58067/new/ https:/

[PATCH] D57898: CodeGen: Fix PR40605: split constant structures generated by -ftrivial-auto-var-init when emitting initializators

glider updated this revision to Diff 186469. glider marked an inline comment as done. glider added a subscriber: pcc. glider added a comment. Added a helper function to decide whether we want to break the structure or not. Right now it only checks for optimization level and structure size. The co

[PATCH] D54978: Move the SMT API to LLVM

brzycki added a comment. In D54978#1394613 , @thakis wrote: > Got it now, sorry about being dense. No problem, I appreciate you looking into this. :) I hope to have some time in the next few days to help out and debug this further. Repository: rC C

[PATCH] D58126: [clangd] Fix a lit-test.

kadircet created this revision. kadircet added a reviewer: sammccall. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ioeric, ilya-biryukov. Herald added a project: clang. Fixes https://bugs.llvm.org/show_bug.cgi?id=40593. Non-percent-encoded chars doesn't cause any problems on

[PATCH] D45978: dllexport const variables must have external linkage.

aaron.ballman added inline comments. Comment at: test/Sema/dllexport-1.c:1 +// RUN: %clang_cc1 -triple i686-win32 -emit-llvm -fms-extensions -std=c99 < %s| FileCheck %s +// RUN: %clang_cc1 -triple x86_64-win32 -emit-llvm -fms-extensions -std=c11 < %s | FileCheck %s

[PATCH] D45978: dllexport const variables must have external linkage.

aaron.ballman added a comment. Oops, sorry, I think I added some comments to a previous revision of the review. To recap: 1. Remove the CHECK line from test/Sema/dllexport-1.cpp 2. Remove the second RUN line from test/Sema/dllexport-1.cpp and test/Sema/dllexport-2.cpp 3. Possibly remove the tri

[PATCH] D58065: [analyzer] Document the frontend library

Szelethus added a comment. Herald added a subscriber: jdoerfert. Please, in the first round of reviews, if you could make high level comments about what should or should not be here would be great, perfectly wrapping around the 80 column limit and finding typos don't concern me as much before t

[PATCH] D58126: [clangd] Fix a lit-test.

sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. Hmm, this removes a useful assertion, and still leaves us sending invalid (or at least non-canonical) URIs in the input. But I don't see a better fix really - lit tests are painful :-(

r353853 - [CMake][Fuchsia] Pass -ldl -lpthread LDFLAGS to second stage

Author: phosek Date: Tue Feb 12 08:24:46 2019 New Revision: 353853 URL: http://llvm.org/viewvc/llvm-project?rev=353853&view=rev Log: [CMake][Fuchsia] Pass -ldl -lpthread LDFLAGS to second stage We're using static libc++ for the second stage which requires explicitly linking -ldl and -lpthread. M

[PATCH] D58128: [PowerPC] Stop defining _ARCH_PWR6X on POWER7 and up

hubert.reinterpretcast created this revision. hubert.reinterpretcast added reviewers: echristo, hfinkel, kbarton, nemanjai, wschmidt. Herald added a subscriber: jsji. Herald added a project: clang. The predefined macro `_ARCH_PWR6X` is associated with GCC's `-mcpu=power6x` option, which enables

[PATCH] D58133: [clangd] Testcase for bug 39811

kadircet created this revision. Herald added subscribers: cfe-commits, jdoerfert, arphaman, jkorous, MaskRay, ioeric, ilya-biryukov. Herald added a project: clang. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D58133 Files: unittests/clangd/XRefsTests.cpp Index: unittests/cl

[PATCH] D58121: [analyzer][WIP] Attempt to fix traversing bindings of non-base regions in ClusterAnalysis

xazax.hun updated this revision to Diff 186489. xazax.hun marked an inline comment as done. xazax.hun edited the summary of this revision. xazax.hun added a comment. Herald added a subscriber: jdoerfert. - Fix test failures. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58121/new/ https

[clang-tools-extra] r353857 - [clangd] Fix a lit-test.

Author: kadircet Date: Tue Feb 12 08:54:47 2019 New Revision: 353857 URL: http://llvm.org/viewvc/llvm-project?rev=353857&view=rev Log: [clangd] Fix a lit-test. Summary: Fixes https://bugs.llvm.org/show_bug.cgi?id=40593. Non-percent-encoded chars doesn't cause any problems on the input-side since

[PATCH] D58126: [clangd] Fix a lit-test.

This revision was automatically updated to reflect the committed changes. Closed by commit rCTE353857: [clangd] Fix a lit-test. (authored by kadircet, committed by ). Changed prior to commit: https://reviews.llvm.org/D58126?vs=186475&id=186490#toc Repository: rCTE Clang Tools Extra CHANGES

[PATCH] D58134: [Analysis] -Wunreachable-code shouldn't fire on the increment of a foreach loop

sammccall created this revision. sammccall added reviewers: ilya-biryukov, ioeric. Herald added a project: clang. Herald added a subscriber: cfe-commits. The idea is that the code here isn't written, so doesn't indicate a bug. Similar to code expanded from macros. This means the warning no longer

[PATCH] D57898: CodeGen: Fix PR40605: split constant structures generated by -ftrivial-auto-var-init when emitting initializators

glider added a comment. This patch shaved off 2.5% of performance slowdown on https://github.com/kamalmarhubi/linux-ipc-benchmarks/blob/master/af_inet_loopback.c on a kernel built in pattern-initialization mode. The .rodata size went down by only 12K (0.5%, since we now split only structures sm

[PATCH] D58074: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier

lildmh marked 11 inline comments as done. lildmh added a comment. Hi Alexey, Thanks very much for your quick review! For the codegen, currently I'm thinking to do it within declare target codegen functions. So there is not a very clear interface to do mapper codegen (also, there is not a clear

[PATCH] D58074: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier

lildmh updated this revision to Diff 186492. lildmh marked 3 inline comments as done. lildmh added a comment. Fix part of Alexey's comments CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58074/new/ https://reviews.llvm.org/D58074 Files: include/clang/AST/OpenMPClause.h include/clang

[PATCH] D58128: [PowerPC] Stop defining _ARCH_PWR6X on POWER7 and up

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

[PATCH] D58135: [clangd] Handle a few more diag kinds in include fixer.

ioeric created this revision. ioeric added a reviewer: sammccall. Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D58135 Files: clangd/IncludeFixer.cpp un

[PATCH] D57898: CodeGen: Fix PR40605: split constant structures generated by -ftrivial-auto-var-init when emitting initializators

jfb added a comment. Overall this LGTM besides a few nits, and wanting input from @rjmccall. As follow-ups (which I can take on): - Handle the case where the types don't match (because `Loc` was adjusted). - Handle small arrays (and any other cases where the struct stores get broken down into c

[PATCH] D58135: [clangd] Handle a few more diag kinds in include fixer.

sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. Nice! Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58135/new/ https://reviews.llvm.org/D58135 _

[PATCH] D58074: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier

ABataev added a comment. In D58074#1394993 , @lildmh wrote: > Hi Alexey, > > Thanks very much for your quick review! > > For the codegen, currently I'm thinking to do it within declare target > codegen functions. So there is not a very clear interface to

[PATCH] D58074: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier

lildmh marked 2 inline comments as done. lildmh added inline comments. Comment at: include/clang/AST/OpenMPClause.h:4236 + static OMPMapClause * + Create(const ASTContext &C, SourceLocation StartLoc, SourceLocation LParenLoc, + SourceLocation EndLoc, ArrayRef Vars, ---

[PATCH] D56305: [AArch64] Support reserving arbitrary general purpose registers

trong added inline comments. Herald added a project: LLVM. Comment at: clang/lib/Driver/ToolChains/Arch/AArch64.cpp:318 + if (Args.hasArg(options::OPT_ffixed_x0)) +Features.push_back("+reserve-x0"); phosek wrote: > trong wrote: > > What happens (should hap

[PATCH] D58074: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier

ABataev added inline comments. Comment at: lib/Sema/SemaLookup.cpp:1074 + if (NameKind == LookupOMPMapperName) { +// Skip out-of-scope declarations. lildmh wrote: > ABataev wrote: > > lildmh wrote: > > > ABataev wrote: > > > > Why do we need special proces

[PATCH] D57232: [ASTImporter] Check visibility/linkage of functions and variables

shafik accepted this revision. shafik added a comment. This revision is now accepted and ready to land. Herald added a subscriber: jdoerfert. I ran `check-lldb` locally and it looks good. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57232/new/ https://reviews.ll

[PATCH] D57736: [WebAssembly] Bulk memory intrinsics and builtins

tlively updated this revision to Diff 186504. tlively added a comment. - Update tests to check full polymorphism of llvm.memcpy and llvm.memmove Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57736/new/ https://reviews.llvm.org/D57736 Files: clan

[PATCH] D56305: [AArch64] Support reserving arbitrary general purpose registers

trong accepted this revision. trong added inline comments. This revision is now accepted and ready to land. Comment at: clang/lib/Driver/ToolChains/Arch/AArch64.cpp:318 + if (Args.hasArg(options::OPT_ffixed_x0)) +Features.push_back("+reserve-x0"); trong wr

[PATCH] D58074: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier

lildmh marked an inline comment as done. lildmh added inline comments. Comment at: lib/Sema/SemaLookup.cpp:1074 + if (NameKind == LookupOMPMapperName) { +// Skip out-of-scope declarations. ABataev wrote: > lildmh wrote: > > ABataev wrote: > > > lildmh wrot

[PATCH] D58074: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier

ABataev added inline comments. Comment at: lib/Sema/SemaLookup.cpp:1074 + if (NameKind == LookupOMPMapperName) { +// Skip out-of-scope declarations. lildmh wrote: > ABataev wrote: > > lildmh wrote: > > > ABataev wrote: > > > > lildmh wrote: > > > > > ABata

[PATCH] D58137: [clang-tidy] Add the abseil-time-subtraction check

hwright created this revision. hwright added reviewers: ioeric, hokein, JonasToth, aaron.ballman. hwright added a project: clang-tools-extra. Herald added subscribers: cfe-commits, jdoerfert, xazax.hun, mgorny. Herald added a project: clang. Repository: rCTE Clang Tools Extra https://reviews.ll

[PATCH] D54978: Move the SMT API to LLVM

brzycki added a comment. The following patch: diff --git a/llvm/cmake/modules/CrossCompile.cmake b/llvm/cmake/modules/CrossCompile.cmake index bc3b210f018..0c30b88f80f 100644 --- a/llvm/cmake/modules/CrossCompile.cmake +++ b/llvm/cmake/modules/CrossCompile.cmake @@ -53,6 +53,7 @@ funct

r353870 - [AMDGPU] Require at least protected visibility for certain symbols

Author: scott.linder Date: Tue Feb 12 10:30:38 2019 New Revision: 353870 URL: http://llvm.org/viewvc/llvm-project?rev=353870&view=rev Log: [AMDGPU] Require at least protected visibility for certain symbols This allows the global visibility controls to be restrictive while still populating the dyn

Re: [PATCH] D57984: PR40642: Fix determination of whether the final statement of a statementexpression is a discarded-value expression.

On Tue, 12 Feb 2019, 05:28 Aaron Ballman via Phabricator via cfe-commits, < cfe-commits@lists.llvm.org> wrote: > aaron.ballman added a comment. > > Considering that this has been fertile ground for buggy edge cases, should > we revert my original changes from the 8.0 branch to give this more time

[PATCH] D56871: [AMDGPU] Require at least protected visibility for certain symbols

This revision was automatically updated to reflect the committed changes. Closed by commit rL353870: [AMDGPU] Require at least protected visibility for certain symbols (authored by scott.linder, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior t

[PATCH] D58107: [MinGW] Add the profiling library when necessary

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

[PATCH] D58106: [compiler-rt] [profile] Provide lprofGetHostName for all windows environments

rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land. lgtm Repository: rCRT Compiler Runtime CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58106/new/ https://reviews.llvm.org/D58106 ___ cfe-com

[PATCH] D57662: [clang-tidy] Parallelise clang-tidy-diff.py

zinovy.nis updated this revision to Diff 186514. zinovy.nis added a comment. `-j` and `-export-fixes` were made mutually exclusive. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57662/new/ https://reviews.llvm.org/D57662 Files: clang-tidy/tool/clang-tidy-diff.py Index: clang-tidy/to

[PATCH] D58074: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier

lildmh marked an inline comment as done. lildmh added inline comments. Comment at: lib/Sema/SemaLookup.cpp:1074 + if (NameKind == LookupOMPMapperName) { +// Skip out-of-scope declarations. ABataev wrote: > lildmh wrote: > > ABataev wrote: > > > lildmh wrot

[PATCH] D58074: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier

ABataev added inline comments. Comment at: lib/Sema/SemaLookup.cpp:1074 + if (NameKind == LookupOMPMapperName) { +// Skip out-of-scope declarations. lildmh wrote: > ABataev wrote: > > lildmh wrote: > > > ABataev wrote: > > > > lildmh wrote: > > > > > ABata

[PATCH] D57977: [HIP] compile option code-object-v3 propagate to llc

ashi1 updated this revision to Diff 186519. ashi1 added a comment. Herald added a subscriber: jdoerfert. Changed patch to us handleTargetFeatures, to consume all -m options passed into clang. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57977/new/ https://reviews.llvm.org/D57977 File

[PATCH] D57977: [HIP] compile option code-object-v3 propagate to llc

kzhuravl added a comment. Can you add some tests? Preferably with -mxnack/-mno-xnack, -msram-ecc/-mno-sram-ecc, -mcode-object-v3/-mno-code-object-v3 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57977/new/ https://reviews.llvm.org/D57977 _

[PATCH] D57991: [Driver][Darwin] Emit an error when using -pg on OS without support for it.

vsapsai updated this revision to Diff 186522. vsapsai added a comment. Herald added a subscriber: jdoerfert. - Use `%select` per Steven's recommendation. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57991/new/ https://reviews.llvm.org/D57991 Files: clang/include/clang/Basic/Diagnost

[PATCH] D58074: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier

kkwli0 added inline comments. Comment at: include/clang/AST/OpenMPClause.h:4077 + /// C++ nested name specifier for the assoicated user-defined mapper. + NestedNameSpecifierLoc MapperQualifierLoc; assoicated -> associated Comment at: includ

[PATCH] D58062: Support framework import/include auto-completion

dgoldman updated this revision to Diff 186524. dgoldman marked 9 inline comments as done. dgoldman added a comment. Herald added a subscriber: jdoerfert. - Misc fixes Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58062/new/ https://reviews.llvm.org/D58062 Files:

[PATCH] D58072: Make ModuleDependencyCollector's method virtual (NFC)

bruno added a comment. I missed that the `DependencyCollector` already marks them virtual, you are just making it obvious here. I think you can omit the ones that are already virtual and only add to the ones that are on the intend of this patch. Repository: rC Clang CHANGES SINCE LAST ACTI

[PATCH] D58137: [clang-tidy] Add the abseil-time-subtraction check

Eugene.Zelenko added inline comments. Comment at: docs/ReleaseNotes.rst:91 + + Finds and fixes `absl::Time` subtraction expressions to do subtraction + in the Time domain instead of the numeric domain. Please use two not one ` for language constructs. ===

[PATCH] D57961: [X86] Add explicit alignment to __m128/__m128i/__m128d/etc. to allow matching of MSVC behavior with #pragma pack.

rnk added a comment. Herald added a subscriber: jdoerfert. Hm, looks like this broke a bunch of code in chromium: bad build: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/ToTWin%28dbg%29/2342 good build: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/ToTWin%28dbg%29/

[PATCH] D57961: [X86] Add explicit alignment to __m128/__m128i/__m128d/etc. to allow matching of MSVC behavior with #pragma pack.

rnk added a comment. I audited all the intrin header uses of __packed__, and I think this was the only one that was missing, so I'll try to fix forward. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57961/new/ https://reviews.llvm.org/D57961 _

[PATCH] D54978: Move the SMT API to LLVM

brzycki added a comment. I think I found why others are struggling to recreate this issue. I did not have the package `z3/bionic` installed. This provides the `/usr/bin/z3` executable which `llvm/cmake/modules/FindZ3.cmake` relies upon to determine the correct `Z3_VERSION_STRING`. If I perform

r353875 - [NFC] typo

Author: jfb Date: Tue Feb 12 12:19:16 2019 New Revision: 353875 URL: http://llvm.org/viewvc/llvm-project?rev=353875&view=rev Log: [NFC] typo Modified: cfe/trunk/include/clang/AST/Decl.h Modified: cfe/trunk/include/clang/AST/Decl.h URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/c

[PATCH] D57977: [HIP] compile option code-object-v3 propagate to llc

ashi1 updated this revision to Diff 186528. ashi1 added a comment. I've added a lit test to check for options such as -mxnack/-mno-xnack, -msram-ecc/-mno-sram-ecc, -mcode-object-v3/-mno-code-object-v3 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57977/new/ https://reviews.llvm.org/D57

[PATCH] D54978: Move the SMT API to LLVM

ddcc added a comment. In D54978#1395268 , @brzycki wrote: > I think I found why others are struggling to recreate this issue. I did not > have the package `z3/bionic` installed. This provides the `/usr/bin/z3` > executable which `llvm/cmake/modules/FindZ

[PATCH] D57977: [HIP] compile option code-object-v3 propagate to llc

kzhuravl added a comment. Is it still code object v2 by default? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57977/new/ https://reviews.llvm.org/D57977 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin

[PATCH] D57977: [HIP] compile option code-object-v3 propagate to llc

ashi1 added a comment. Yes, but it will need this PR: https://github.com/ROCm-Developer-Tools/HIP/pull/910 So that the default is set. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57977/new/ https://reviews.llvm.org/D57977 ___ cfe-commits

[PATCH] D57977: [HIP] compile option code-object-v3 propagate to llc

kzhuravl accepted this revision. kzhuravl added a comment. This revision is now accepted and ready to land. LGTM. Please make sure to update your commit message as your patch processes all m options now. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57977/new/ https://reviews.llvm.org/

r353877 - Disable PIC/PIE for MSP430 target

Author: asl Date: Tue Feb 12 12:46:00 2019 New Revision: 353877 URL: http://llvm.org/viewvc/llvm-project?rev=353877&view=rev Log: Disable PIC/PIE for MSP430 target Relocatable code generation is meaningless on MSP430, as the platform is too small to use shared libraries. Patch by Dmitry Mikushe

[PATCH] D56927: Disable PIC/PIE for MSP430 target

This revision was automatically updated to reflect the committed changes. Closed by commit rL353877: Disable PIC/PIE for MSP430 target (authored by asl, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Repository: rL LLVM CHANGES SINCE LAST ACTION https:

[PATCH] D58072: Make ModuleDependencyCollector's method virtual (NFC)

JDevlieghere added a comment. Alright, I'll split this up in two patches Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58072/new/ https://reviews.llvm.org/D58072 ___ cfe-commits mailing list cfe-commits@lists.llvm.or

r353878 - [X86] Use __m128_u for _mm_loadu_ps after r353555

Author: rnk Date: Tue Feb 12 13:04:21 2019 New Revision: 353878 URL: http://llvm.org/viewvc/llvm-project?rev=353878&view=rev Log: [X86] Use __m128_u for _mm_loadu_ps after r353555 Add secondary triple to existing SSE test for it. I audited other uses of __attribute__((__packed__)) in the intrins

  1   2   >