[clang] a7db230 - [X86] Add CMPXCHG16B feature to amdfam10 in the frontend.

2020-06-25 Thread Craig Topper via cfe-commits
Author: Craig Topper Date: 2020-06-25T22:55:36-07:00 New Revision: a7db230d752be4a104c496eb68a82a5acb4bb35c URL: https://github.com/llvm/llvm-project/commit/a7db230d752be4a104c496eb68a82a5acb4bb35c DIFF: https://github.com/llvm/llvm-project/commit/a7db230d752be4a104c496eb68a82a5acb4bb35c.diff

[PATCH] D82506: [HIP] Add missing options for lto

2020-06-25 Thread Yaxun Liu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG8013ce449022: [HIP] Add missing options for lto (authored by yaxunl). Herald added a project: clang. Changed prior to commit: https://reviews.llvm.org/D82506?vs=273424=273588#toc Repository: rG LLVM

[PATCH] D82312: Add `CharLiteral` to SyntaxTree

2020-06-25 Thread Vlad Vereschaka via Phabricator via cfe-commits
vvereschaka added a comment. Hello Eduardo, sorry, but one of your these commits break the clang unit tests on the following builders: - http://lab.llvm.org:8011/builders/llvm-clang-x86_64-win-fast/builds/18313 - http://lab.llvm.org:8011/builders/llvm-clang-win-x-aarch64/builds/256 -

[clang] 8013ce4 - [HIP] Add missing options for lto

2020-06-25 Thread Yaxun Liu via cfe-commits
Author: Yaxun (Sam) Liu Date: 2020-06-26T00:26:05-04:00 New Revision: 8013ce4490225faf8aae911ee88087ce445e604c URL: https://github.com/llvm/llvm-project/commit/8013ce4490225faf8aae911ee88087ce445e604c DIFF:

[clang] 471c806 - [hip] Refine `clang/test/CodeGenCUDA/amdgpu-kernel-arg-pointer-type.cu`

2020-06-25 Thread Michael Liao via cfe-commits
Author: Michael Liao Date: 2020-06-25T23:57:08-04:00 New Revision: 471c806a45bbac2f0f4274d8bea383d06d397a84 URL: https://github.com/llvm/llvm-project/commit/471c806a45bbac2f0f4274d8bea383d06d397a84 DIFF: https://github.com/llvm/llvm-project/commit/471c806a45bbac2f0f4274d8bea383d06d397a84.diff

[PATCH] D82579: [NFC] Extract unifyTargetFeatures

2020-06-25 Thread Yaxun Liu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. yaxunl marked an inline comment as done. Closed by commit rGed398c3ca404: [NFC] Extract unifyTargetFeatures (authored by yaxunl). Herald added a project: clang. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D82623: [sve][acle] Enable feature macros for SVE ACLE extensions.

2020-06-25 Thread Francesco Petrogalli via Phabricator via cfe-commits
fpetrogalli added a comment. Reviewers, I have added 2 parent revision with the last two set of intrinsics that are enabled by the macro introduced in this patch. I will update those tests in this patch once the patches are in. Meanwhile, please double check that my interpretation of the

[PATCH] D82623: [sve][acle] Enable feature macros for SVE ACLE extensions.

2020-06-25 Thread Francesco Petrogalli via Phabricator via cfe-commits
fpetrogalli created this revision. fpetrogalli added reviewers: sdesmalen, efriedma, c-rhodes, kmclaughlin, SjoerdMeijer. Herald added subscribers: cfe-commits, psnobl, rkruppe, kristof.beyls, tschuett. Herald added a reviewer: rengolin. Herald added a project: clang. fpetrogalli added parent

[clang] ed398c3 - [NFC] Extract unifyTargetFeatures

2020-06-25 Thread Yaxun Liu via cfe-commits
Author: Yaxun (Sam) Liu Date: 2020-06-25T23:17:08-04:00 New Revision: ed398c3ca404f9ec50b134863a227555a9e374df URL: https://github.com/llvm/llvm-project/commit/ed398c3ca404f9ec50b134863a227555a9e374df DIFF:

[PATCH] D82620: [clang-format] Preserve whitespace in selected macros

2020-06-25 Thread Jake Merdich via Phabricator via cfe-commits
JakeMerdichAMD created this revision. Herald added subscribers: cfe-commits, kristof.beyls. Herald added a project: clang. JakeMerdichAMD added reviewers: MyDeveloperDay, curdeius, sammccall, jbcoe. JakeMerdichAMD added a project: clang-format. https://bugs.llvm.org/show_bug.cgi?id=46383 When

[PATCH] D80970: [PowerPC][Power10] Implement centrifuge, vector gather every nth bit, vector evaluate Builtins in LLVM/Clang

2020-06-25 Thread Amy Kwan via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGe0c02dc9800e: [PowerPC][Power10] Implement centrifuge, vector gather every nth bit, vector… (authored by amyk). Changed prior to commit: https://reviews.llvm.org/D80970?vs=267775=273581#toc

[clang] e0c02dc - [PowerPC][Power10] Implement centrifuge, vector gather every nth bit, vector evaluate Builtins in LLVM/Clang

2020-06-25 Thread Amy Kwan via cfe-commits
Author: Amy Kwan Date: 2020-06-25T21:34:41-05:00 New Revision: e0c02dc9800ebd317d1369848f4e74c8f783533a URL: https://github.com/llvm/llvm-project/commit/e0c02dc9800ebd317d1369848f4e74c8f783533a DIFF: https://github.com/llvm/llvm-project/commit/e0c02dc9800ebd317d1369848f4e74c8f783533a.diff

[clang] 0723b18 - [hip] Re-enable `clang/test/CodeGenCUDA/amdgpu-kernel-arg-pointer-type.cu`

2020-06-25 Thread Michael Liao via cfe-commits
Author: Michael Liao Date: 2020-06-25T22:29:27-04:00 New Revision: 0723b1891fac8f79f92549e3bcac9112be4ebd43 URL: https://github.com/llvm/llvm-project/commit/0723b1891fac8f79f92549e3bcac9112be4ebd43 DIFF: https://github.com/llvm/llvm-project/commit/0723b1891fac8f79f92549e3bcac9112be4ebd43.diff

[PATCH] D82609: [PowerPC][Power10] Implement Vector Multiply High/Divide Extended Builtins in LLVM/Clang

2020-06-25 Thread Amy Kwan via Phabricator via cfe-commits
amyk updated this revision to Diff 273578. amyk added a comment. Addressed Anil's comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82609/new/ https://reviews.llvm.org/D82609 Files: clang/include/clang/Basic/BuiltinsPPC.def clang/lib/Headers/altivec.h

[PATCH] D79773: [clang-format] Improve clang-formats handling of concepts

2020-06-25 Thread Johel Ernesto Guerrero Peña via Phabricator via cfe-commits
JohelEGP requested changes to this revision. JohelEGP added a comment. This revision now requires changes to proceed. `BreakBeforeBraces: Allman` isn't respected. template friend constexpr auto operator*(const size2d& l, const units::quantity& r) noexcept( noexcept(l.w* r))

[clang] d3f437d - [hip] Disable test temporarily due to failures on build servers.

2020-06-25 Thread Michael Liao via cfe-commits
Author: Michael Liao Date: 2020-06-25T22:04:20-04:00 New Revision: d3f437d35189f7567294daf3e60e08326e64994a URL: https://github.com/llvm/llvm-project/commit/d3f437d35189f7567294daf3e60e08326e64994a DIFF: https://github.com/llvm/llvm-project/commit/d3f437d35189f7567294daf3e60e08326e64994a.diff

[PATCH] D82604: No nested namespaces in Clang-Tidy checkers

2020-06-25 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay requested changes to this revision. MaskRay added a comment. This revision now requires changes to proceed. As the warnings suggest, nested name definitions can't be used because llvm-project is still using C++14. `llvm.org` as a reviewer doesn't work. You can use `git log` and find

[PATCH] D82617: Disable GCC's -Woverloaded-virtual, which has false positives.

2020-06-25 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added reviewers: kadircet, uabelho, bjope. Herald added subscribers: cfe-commits, mgorny. Herald added a project: clang. Currently this warning is on for both clang and GCC, when building clang. It's off for the rest of LLVM, so my sense is it isn't that

[PATCH] D79732: AMDGPU/HIP: Don't replace pointer types in kernel argument structs

2020-06-25 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm abandoned this revision. arsenm added a comment. I think this is obsoleted now CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79732/new/ https://reviews.llvm.org/D79732 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

LLVM Lab network works today at 6:30 PM PST

2020-06-25 Thread Galina Kistanova via cfe-commits
Hello everyone, LLVM Lab will be unavailable for a short time at about 6:30 PM PST due network maintenance. Thank you for understanding. Thanks Galina ___ cfe-commits mailing list cfe-commits@lists.llvm.org

RE: [clang] 466e8b7 - Add StringLiteral to SyntaxTree

2020-06-25 Thread Yung, Douglas via cfe-commits
Hi Eduardo, Your change is causing a build failure on PS4 Windows build bot, can you please take a look and fix it or revert the change if you cannot fix it soon so we can get the bot green again? http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast/builds/33164

[PATCH] D81938: [InferAddressSpaces] Handle the pair of `ptrtoint`/`inttoptr`.

2020-06-25 Thread Michael Liao via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGdccfaacf93e1: [InferAddressSpaces] Handle the pair of `ptrtoint`/`inttoptr`. (authored by hliao). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81938/new/

[PATCH] D81315: [analyzer] Warning for default constructed unique pointer dereferences

2020-06-25 Thread Nithin VR via Phabricator via cfe-commits
vrnithinkumar added a comment. In D81315#2079457 , @Szelethus wrote: > Best of luck on your GSoC! I don't have much else to add to your patch, but > you seem to have made good progress already! Thanks! > In D81315#2078043

[clang-tools-extra] 0e1997e - [clangd] Fix test compile with GCC (name conflict)

2020-06-25 Thread Sam McCall via cfe-commits
Author: Sam McCall Date: 2020-06-26T03:02:28+02:00 New Revision: 0e1997ed4ef74202a1bec6e7c18ba7a1df496896 URL: https://github.com/llvm/llvm-project/commit/0e1997ed4ef74202a1bec6e7c18ba7a1df496896 DIFF: https://github.com/llvm/llvm-project/commit/0e1997ed4ef74202a1bec6e7c18ba7a1df496896.diff

[PATCH] D81315: [analyzer] Warning for default constructed unique pointer dereferences

2020-06-25 Thread Nithin VR via Phabricator via cfe-commits
vrnithinkumar updated this revision to Diff 273568. vrnithinkumar marked 16 inline comments as done. vrnithinkumar added a comment. Addressing review comments CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81315/new/ https://reviews.llvm.org/D81315 Files:

[PATCH] D81315: [analyzer] Warning for default constructed unique pointer dereferences

2020-06-25 Thread Nithin VR via Phabricator via cfe-commits
vrnithinkumar added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/SmartPtrModeling.cpp:35 bool isNullAfterMoveMethod(const CallEvent ) const; + BugType NullDereferenceBugType{this, "Null-smartPtr-deref", + "C++ smart

[PATCH] D82249: [HWASan] Disable GlobalISel/FastISel for HWASan Globals.

2020-06-25 Thread Mitch Phillips via Phabricator via cfe-commits
hctim abandoned this revision. hctim added a comment. Abandoning - fixing the underlying issue at D82615 . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82249/new/ https://reviews.llvm.org/D82249

[clang] dccfaac - [InferAddressSpaces] Handle the pair of `ptrtoint`/`inttoptr`.

2020-06-25 Thread Michael Liao via cfe-commits
Author: Michael Liao Date: 2020-06-25T20:46:56-04:00 New Revision: dccfaacf93e1c4801cbcc4686f64eb8a35564ff7 URL: https://github.com/llvm/llvm-project/commit/dccfaacf93e1c4801cbcc4686f64eb8a35564ff7 DIFF: https://github.com/llvm/llvm-project/commit/dccfaacf93e1c4801cbcc4686f64eb8a35564ff7.diff

[PATCH] D82609: [PowerPC][Power10] Implement Vector Multiply High/Divide Extended Builtins in LLVM/Clang

2020-06-25 Thread Amy Kwan via Phabricator via cfe-commits
amyk marked 2 inline comments as done. amyk added inline comments. Comment at: clang/test/CodeGen/builtins-ppc-p10vector.c:18 + // CHECK-NEXT: ret <4 x i32> + return vec_dive(vsia, vsib); +} anil9 wrote: > I may be wrong but where are the variables declared ?

[PATCH] D82612: [clangd] Config: compile Fragment -> CompiledFragment -> Config

2020-06-25 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. Herald added subscribers: cfe-commits, usaxena95, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov, mgorny. Herald added a project: clang. sammccall added a parent revision: D82606: [clangd] Config: config struct propagated through Context. sammccall edited

[PATCH] D76323: [AST] Fix handling of long double and bool in __builtin_bit_cast

2020-06-25 Thread Erik Pilkington via Phabricator via cfe-commits
erik.pilkington added a comment. Ping, @rsmith did you want to take another look at this? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76323/new/ https://reviews.llvm.org/D76323 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D82611: [SemaObjC] Add a warning for @selector expressions that potentially refer to objc_direct methods

2020-06-25 Thread Erik Pilkington via Phabricator via cfe-commits
erik.pilkington created this revision. erik.pilkington added reviewers: rjmccall, MadCoder. Herald added subscribers: ributzka, dexonsmith, jkorous. As a heuristic, only warn if the selector matches a method declared in the current interface. rdar://64621668 https://reviews.llvm.org/D82611

[PATCH] D82609: [PowerPC][Power10] Implement Vector Multiply High/Divide Extended Builtins in LLVM/Clang

2020-06-25 Thread Anil Mahmud via Phabricator via cfe-commits
anil9 added inline comments. Comment at: clang/test/CodeGen/builtins-ppc-p10vector.c:18 + // CHECK-NEXT: ret <4 x i32> + return vec_dive(vsia, vsib); +} I may be wrong but where are the variables declared ? I do not see the variables delclared above in the

[PATCH] D82130: [WebAssembly] Adding 64-bit versions of __stack_pointer and other globals

2020-06-25 Thread Wouter van Oortmerssen via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGb9a539c01084: [WebAssembly] Adding 64-bit versions of __stack_pointer and other globals (authored by aardappel). Changed prior to commit: https://reviews.llvm.org/D82130?vs=273539=273548#toc

[PATCH] D82502: [PowerPC][Power10] Implement Load VSX Vector and Sign Extend and Zero Extend

2020-06-25 Thread Anil Mahmud via Phabricator via cfe-commits
anil9 added inline comments. Comment at: clang/test/CodeGen/builtins-ppc-p10vector.c:17 +unsigned int uia, *uiap; +signed int *ia; +signed short *sia; nit: It seems that most pull requests follow an ordering like first signed declaration and then unsigned,

[PATCH] D82609: [PowerPC][Power10] Implement Vector Multiply High/Divide Extended Builtins in LLVM/Clang

2020-06-25 Thread Amy Kwan via Phabricator via cfe-commits
amyk created this revision. amyk added reviewers: power-llvm-team, PowerPC, nemanjai, lei, saghir. amyk added projects: LLVM, clang, PowerPC. Herald added subscribers: shchenz, hiraditya. This patch implements the following function prototypes to utilize the `vmulh[s|u][w|d]` and

[PATCH] D82606: [clangd] Config: config struct propagated through Context

2020-06-25 Thread Sam McCall via Phabricator via cfe-commits
sammccall updated this revision to Diff 273543. sammccall added a comment. config() -> Config::current(). "config" already names a namespace. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82606/new/ https://reviews.llvm.org/D82606 Files:

[PATCH] D82130: [WebAssembly] Adding 64-bit versions of __stack_pointer and other globals

2020-06-25 Thread Wouter van Oortmerssen via Phabricator via cfe-commits
aardappel updated this revision to Diff 273539. aardappel added a comment. types & variables in InputChunks.cpp CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82130/new/ https://reviews.llvm.org/D82130 Files: clang/lib/Driver/ToolChains/WebAssembly.cpp lld/wasm/Config.h

[clang] b9a539c - [WebAssembly] Adding 64-bit versions of __stack_pointer and other globals

2020-06-25 Thread Wouter van Oortmerssen via cfe-commits
Author: Wouter van Oortmerssen Date: 2020-06-25T15:52:44-07:00 New Revision: b9a539c01084a572e406579c326a0da1e22e286f URL: https://github.com/llvm/llvm-project/commit/b9a539c01084a572e406579c326a0da1e22e286f DIFF:

[PATCH] D82130: [WebAssembly] Adding 64-bit versions of __stack_pointer and other globals

2020-06-25 Thread Derek Schuff via Phabricator via cfe-commits
dschuff accepted this revision. dschuff added a comment. Compiler changes LGTM Comment at: llvm/test/CodeGen/WebAssembly/stack-alignment.ll:1 -; RUN: llc < %s -asm-verbose=false -disable-wasm-fallthrough-return-opt -wasm-keep-registers | FileCheck %s - -target datalayout =

[PATCH] D82122: [analyzer][Liveness][RFC][NFC] Propose to turn statement liveness into expression liveness

2020-06-25 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus abandoned this revision. Szelethus added a comment. This revision has done its job. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82122/new/ https://reviews.llvm.org/D82122 ___ cfe-commits

[PATCH] D82598: [analyzer][Liveness][NFC] Get rid of statement liveness, because such a thing doesn't exist

2020-06-25 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. > We could just kill all subexpr at the end of the full expression I suspect that it would be pretty bad if you, say, kill the condition of the `if`-statement before picking the branch. Or kill the

[PATCH] D71687: Fix full loop unrolling initialization in new pass manager

2020-06-25 Thread Eric Christopher via Phabricator via cfe-commits
echristo closed this revision. echristo added a comment. Committed a while back. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71687/new/ https://reviews.llvm.org/D71687 ___ cfe-commits mailing list

[PATCH] D81938: [InferAddressSpaces] Handle the pair of `ptrtoint`/`inttoptr`.

2020-06-25 Thread Michael Liao via Phabricator via cfe-commits
hliao updated this revision to Diff 273531. hliao added a comment. remove 'an' Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81938/new/ https://reviews.llvm.org/D81938 Files: clang/test/CodeGenCUDA/amdgpu-kernel-arg-pointer-type.cu

[PATCH] D82598: [analyzer][Liveness][NFC] Get rid of statement liveness, because such a thing doesn't exist

2020-06-25 Thread Gábor Horváth via Phabricator via cfe-commits
xazax.hun added a comment. I always wondered if we actually need to track the liveness of exprs at all. We could just kill all subexpr at the end of the full expression without precomputing anything. But I might miss something. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D78655: [CUDA][HIP] Let non-caputuring lambda be host device

2020-06-25 Thread Aaron Enye Shi via Phabricator via cfe-commits
ashi1 added inline comments. Comment at: clang/test/CodeGenCUDA/lambda.cu:53 +// DEV: call void @_ZZ12test_resolvevENKUlvE_clEv +// DEV-LABE: define internal void @_ZZ12test_resolvevENKUlvE_clEv +// DEV: call i32 @_Z10overloadedIiET_v There is a typo here,

[PATCH] D79719: [AIX] Implement AIX special alignment rule about double/long double

2020-06-25 Thread Hubert Tong via Phabricator via cfe-commits
hubert.reinterpretcast added inline comments. Comment at: clang/include/clang/AST/RecordLayout.h:75 + // can be different than Alignment in cases where it is beneficial for + // performance or backwards compatibility perserving (e.g. AIX-ABI). + CharUnits PreferredAlignment;

[PATCH] D79972: [OpenMP5.0] map item can be non-contiguous for target update

2020-06-25 Thread Chi Chun Chen via Phabricator via cfe-commits
cchen updated this revision to Diff 273529. cchen added a comment. Pass Info directly Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79972/new/ https://reviews.llvm.org/D79972 Files: clang/include/clang/AST/OpenMPClause.h

[PATCH] D82561: [analyzer][CrossTU] Lower CTUImportThreshold default value

2020-06-25 Thread Gábor Horváth via Phabricator via cfe-commits
xazax.hun added a comment. The analyzer inlines small functions within a TU regardless of the thresholds. I think it would be sensible to do the same across TUs in the case we don't do this already. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D82130: [WebAssembly] Adding 64-bit versions of __stack_pointer and other globals

2020-06-25 Thread Sam Clegg via Phabricator via cfe-commits
sbc100 accepted this revision. sbc100 added a comment. Linker changes still lgtm % comments Comment at: lld/wasm/InputChunks.cpp:338 + auto WASM_OPCODE_PTR_CONST = + config->is64 ? WASM_OPCODE_I64_CONST : WASM_OPCODE_I32_CONST; sbc100 wrote: > Just use

[PATCH] D82604: No nested namespaces in Clang-Tidy checkers

2020-06-25 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri added a comment. Isn't that C++17? LLVM is at C++14. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82604/new/ https://reviews.llvm.org/D82604 ___ cfe-commits mailing list

[PATCH] D82606: [clangd] Config: config struct propagated through Context

2020-06-25 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added reviewers: hokein, kadircet. Herald added subscribers: cfe-commits, usaxena95, arphaman, jkorous, MaskRay, ilya-biryukov, mgorny. Herald added a project: clang. This introduces the "semantic form" of config exposed to features, contrasted with the

[PATCH] D81938: [InferAddressSpaces] Handle the pair of `ptrtoint`/`inttoptr`.

2020-06-25 Thread Michael Liao via Phabricator via cfe-commits
hliao updated this revision to Diff 273524. hliao added a comment. Revie the grammar. s/is/as/ in the first sentence. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81938/new/ https://reviews.llvm.org/D81938 Files:

[PATCH] D82226: Add Metadata to Transformer tooling

2020-06-25 Thread Andy Soffer via Phabricator via cfe-commits
asoffer updated this revision to Diff 273522. asoffer added a comment. Last diff was a mistake. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82226/new/ https://reviews.llvm.org/D82226 Files:

[PATCH] D80970: [PowerPC][Power10] Implement centrifuge, vector gather every nth bit, vector evaluate Builtins in LLVM/Clang

2020-06-25 Thread Amy Kwan via Phabricator via cfe-commits
amyk added a comment. Talked to Anil, and will address his nit comments on the commit and disregard the comment about the test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80970/new/ https://reviews.llvm.org/D80970

[PATCH] D82604: No nested namespaces in Clang-Tidy checkers

2020-06-25 Thread Seija Kijin via Phabricator via cfe-commits
pi1024e created this revision. Herald added subscribers: cfe-commits, Charusso. Herald added a project: clang. A lot of the code, including ironically the nested namespace checker itself, was using "namespace clang { tidy { modernize...", so I put those together for readability as "namespace

[PATCH] D79972: [OpenMP5.0] map item can be non-contiguous for target update

2020-06-25 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:10334-10335 +MCHandler.generateAllInfo(BasePointers, Pointers, Sizes, MapTypes, + Info.Dims, Info.Offsets, Info.Counts, + Info.Strides);

[PATCH] D82226: Add Metadata to Transformer tooling

2020-06-25 Thread Andy Soffer via Phabricator via cfe-commits
asoffer updated this revision to Diff 273520. asoffer marked an inline comment as done. asoffer added a comment. Updating other test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82226/new/ https://reviews.llvm.org/D82226 Files:

[PATCH] D82226: Add Metadata to Transformer tooling

2020-06-25 Thread Andy Soffer via Phabricator via cfe-commits
asoffer updated this revision to Diff 273516. asoffer marked an inline comment as done and an inline comment as not done. asoffer added a comment. Commenting fields and simplifying test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82226/new/

[PATCH] D79972: [OpenMP5.0] map item can be non-contiguous for target update

2020-06-25 Thread Chi Chun Chen via Phabricator via cfe-commits
cchen updated this revision to Diff 273517. cchen added a comment. Fix based on feedback Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79972/new/ https://reviews.llvm.org/D79972 Files: clang/include/clang/AST/OpenMPClause.h

[PATCH] D82579: [NFC] Extract unifyTargetFeatures

2020-06-25 Thread Artem Belevich via Phabricator via cfe-commits
tra accepted this revision. tra added inline comments. This revision is now accepted and ready to land. Comment at: clang/lib/Driver/ToolChains/CommonArgs.cpp:93 +StringRef Name = Features[I]; +assert(Name[0] == '-' || Name[0] == '+'); +LastOpt[Name.drop_front(1)] =

[PATCH] D82561: [analyzer][CrossTU] Lower CTUImportThreshold default value

2020-06-25 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus accepted this revision. Szelethus added a comment. This revision is now accepted and ready to land. Based on discussions I heard in between @dkrupp, @martong and You, this seems appropriate. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D82386: [clangd] Config: Fragments and parsing from YAML

2020-06-25 Thread Sam McCall via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. sammccall marked an inline comment as done. Closed by commit rGe9fb1506b83d: [clangd] Config: Fragments and parsing from YAML (authored by sammccall). Changed prior to commit:

[PATCH] D82226: Add Metadata to Transformer tooling

2020-06-25 Thread Andy Soffer via Phabricator via cfe-commits
asoffer marked 2 inline comments as done. asoffer added a comment. In D82226#2114111 , @ymandel wrote: > Looks good! Only real question is one of design -- should we consider the > (deeper) change of templating the various types rather than using dynamic

[PATCH] D82386: [clangd] Config: Fragments and parsing from YAML

2020-06-25 Thread Sam McCall via Phabricator via cfe-commits
sammccall marked 9 inline comments as done. sammccall added a comment. Thanks for the careful review! Comment at: clang-tools-extra/clangd/ConfigFragment.h:73 +/// Only valid if SourceManager is set. +llvm::SMLoc Location; + }; kadircet wrote: >

[PATCH] D82598: [analyzer][Liveness][NFC] Get rid of statement liveness, because such a thing doesn't exist

2020-06-25 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus created this revision. Szelethus added reviewers: NoQ, xazax.hun, vsavchenko, martong, dcoughlin. Szelethus added a project: clang. Herald added subscribers: cfe-commits, ASDenysPetrov, steakhal, Charusso, gamesh411, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, rnkovacs, szepet,

[PATCH] D82598: [analyzer][Liveness][NFC] Get rid of statement liveness, because such a thing doesn't exist

2020-06-25 Thread Kristóf Umann via Phabricator via cfe-commits
Szelethus added a comment. I didn't run this on big projects just yet, and judging from the fact that `Environment` only contained ObjetiveC examples of non-expression statements, I might need a tip on how to test on ObjC code :) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D81938: [InferAddressSpaces] Handle the pair of `ptrtoint`/`inttoptr`.

2020-06-25 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm added inline comments. Comment at: llvm/lib/Transforms/Scalar/InferAddressSpaces.cpp:238-239 +return false; + // Check it's really safe to treat that pair of `ptrtoint`/`inttoptr` is a + // no-op cast. Besides checking both of them are no-op casts, as the + //

[PATCH] D79972: [OpenMP5.0] map item can be non-contiguous for target update

2020-06-25 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev added inline comments. Comment at: clang/lib/Sema/SemaOpenMP.cpp:16624 + if (IsPointer && !AllowAnotherPtr) +SemaRef.Diag(ELoc, diag::err_omp_section_length_undefined) << true; + else cchen wrote: > ABataev wrote: > > Better to use

[PATCH] D82130: [WebAssembly] Adding 64-bit versions of __stack_pointer and other globals

2020-06-25 Thread Wouter van Oortmerssen via Phabricator via cfe-commits
aardappel updated this revision to Diff 273503. aardappel added a comment. - Fixed ISEL for FrameIndex - Fixed 64-bit conditions in branches (thanks @aheejin!) - Made the FrameIndex generation code in WebAssemblyRegisterInfo work. - Made userstack.ll and stack-alignment.ll pass in wasm64. - Code

[clang-tools-extra] e9fb150 - [clangd] Config: Fragments and parsing from YAML

2020-06-25 Thread Sam McCall via cfe-commits
Author: Sam McCall Date: 2020-06-25T22:55:45+02:00 New Revision: e9fb1506b83d001082dc535af12acf45050a527c URL: https://github.com/llvm/llvm-project/commit/e9fb1506b83d001082dc535af12acf45050a527c DIFF: https://github.com/llvm/llvm-project/commit/e9fb1506b83d001082dc535af12acf45050a527c.diff

[PATCH] D79972: [OpenMP5.0] map item can be non-contiguous for target update

2020-06-25 Thread Chi Chun Chen via Phabricator via cfe-commits
cchen marked an inline comment as done. cchen added inline comments. Comment at: clang/lib/Sema/SemaOpenMP.cpp:16624 + if (IsPointer && !AllowAnotherPtr) +SemaRef.Diag(ELoc, diag::err_omp_section_length_undefined) << true; + else ABataev wrote:

[PATCH] D82345: [sve][acle] Implement some of the C intrinsics for brain float.

2020-06-25 Thread Francesco Petrogalli via Phabricator via cfe-commits
fpetrogalli updated this revision to Diff 273498. fpetrogalli marked 5 inline comments as done. fpetrogalli added a comment. This patch needed some love... @c-rhodes, I have addressed your feedback, thank you. I have also predicated all the instruction selection pattern for `-mattr=+bf16`, and

[PATCH] D82579: [NFC] Extract unifyTargetFeatures

2020-06-25 Thread Yaxun Liu via Phabricator via cfe-commits
yaxunl marked 3 inline comments as done. yaxunl added inline comments. Comment at: clang/lib/Driver/ToolChains/CommonArgs.cpp:93 +StringRef Name = Features[I]; +assert(Name[0] == '-' || Name[0] == '+'); +LastOpt[Name.drop_front(1)] = I; tra wrote: >

[PATCH] D79972: [OpenMP5.0] map item can be non-contiguous for target update

2020-06-25 Thread Chi Chun Chen via Phabricator via cfe-commits
cchen marked an inline comment as done. cchen added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:8781 MappableExprsHandler::MapFlagsArrayTy , - CGOpenMPRuntime::TargetDataInfo ) { +

[PATCH] D81282: [builtins] Move more float128-related helpers to GENERIC_TF_SOURCES list

2020-06-25 Thread Anton Korobeynikov via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGa931ec7ca0ee: [builtins] Move more float128-related helpers to GENERIC_TF_SOURCES list (authored by atrosinenko, committed by asl). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D79972: [OpenMP5.0] map item can be non-contiguous for target update

2020-06-25 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev added inline comments. Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:8781 MappableExprsHandler::MapFlagsArrayTy , - CGOpenMPRuntime::TargetDataInfo ) { + MappableExprsHandler::MapDimArrayTy , +

[PATCH] D82563: [Sema][NFC] Remove Redundant Condition

2020-06-25 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. Nice! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82563/new/ https://reviews.llvm.org/D82563

[PATCH] D79972: [OpenMP5.0] map item can be non-contiguous for target update

2020-06-25 Thread Chi Chun Chen via Phabricator via cfe-commits
cchen updated this revision to Diff 273484. cchen added a comment. Fix coding style Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79972/new/ https://reviews.llvm.org/D79972 Files: clang/include/clang/AST/OpenMPClause.h

[PATCH] D82592: [libTooling] Rename overloaded `range` range selector.

2020-06-25 Thread Yitzhak Mandelbaum via Phabricator via cfe-commits
ymandel created this revision. ymandel added a reviewer: tdl-g. Herald added a project: clang. tdl-g accepted this revision. This revision is now accepted and ready to land. Renames the overloaded `RangeSelector` combinator `range` to the more descriptive `enclose` and `encloseNodes`. The old

[PATCH] D81672: [Driver] When forcing a crash print the bug report message

2020-06-25 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added inline comments. Comment at: clang/tools/driver/driver.cpp:515 + + llvm::dbgs() << llvm::getBugReportMsg(); } john.brawn wrote: > MaskRay wrote: > > Why ` llvm::dbgs() << llvm::getBugReportMsg();` when -gen-reproducer is > > specified?

[clang] 6673d69 - [X86] Don't imply -mprfchw when -m3dnow is specified. Enable prefetchw in the backend with 3dnow feature.

2020-06-25 Thread Craig Topper via cfe-commits
Author: Craig Topper Date: 2020-06-25T12:46:52-07:00 New Revision: 6673d69226d86f4906dca4b627d6e0582486d072 URL: https://github.com/llvm/llvm-project/commit/6673d69226d86f4906dca4b627d6e0582486d072 DIFF: https://github.com/llvm/llvm-project/commit/6673d69226d86f4906dca4b627d6e0582486d072.diff

[PATCH] D82586: [HIP] Improve check patterns to avoid test failures in case string "opt", etc. happens to be in the command path.

2020-06-25 Thread Hiroshi Yamauchi via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG9da93f590414: [HIP] Improve check patterns to avoid test failures in case string opt, etc. (authored by yamauchi). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D82579: [NFC] Extract unifyTargetFeatures

2020-06-25 Thread Artem Belevich via Phabricator via cfe-commits
tra marked an inline comment as done. tra added inline comments. Comment at: clang/lib/Driver/ToolChains/CommonArgs.cpp:93 +StringRef Name = Features[I]; +assert(Name[0] == '-' || Name[0] == '+'); +LastOpt[Name.drop_front(1)] = I; yaxunl wrote: > tra

[PATCH] D79719: [AIX] Implement AIX special alignment rule about double/long double

2020-06-25 Thread Jason Liu via Phabricator via cfe-commits
jasonliu accepted this revision. jasonliu added a comment. LGTM. I have no further comments on this patch. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79719/new/ https://reviews.llvm.org/D79719 ___ cfe-commits mailing list

[PATCH] D81672: [Driver] When forcing a crash print the bug report message

2020-06-25 Thread John Brawn via Phabricator via cfe-commits
john.brawn marked an inline comment as done. john.brawn added inline comments. Comment at: clang/tools/driver/driver.cpp:515 + + llvm::dbgs() << llvm::getBugReportMsg(); } MaskRay wrote: > Why ` llvm::dbgs() << llvm::getBugReportMsg();` when

[PATCH] D81938: [InferAddressSpaces] Handle the pair of `ptrtoint`/`inttoptr`.

2020-06-25 Thread Michael Liao via Phabricator via cfe-commits
hliao updated this revision to Diff 273477. hliao added a comment. Revise Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81938/new/ https://reviews.llvm.org/D81938 Files: clang/test/CodeGenCUDA/amdgpu-kernel-arg-pointer-type.cu

[PATCH] D81282: [builtins] Move more float128-related helpers to GENERIC_TF_SOURCES list

2020-06-25 Thread Eli Friedman via Phabricator via cfe-commits
efriedma accepted this revision. efriedma 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/D81282/new/ https://reviews.llvm.org/D81282

[clang] 9da93f5 - [HIP] Improve check patterns to avoid test failures in case string "opt", etc. happens to be in the command path.

2020-06-25 Thread Hiroshi Yamauchi via cfe-commits
Author: Hiroshi Yamauchi Date: 2020-06-25T12:18:43-07:00 New Revision: 9da93f590414ae3b7be20061d8a97695b263482e URL: https://github.com/llvm/llvm-project/commit/9da93f590414ae3b7be20061d8a97695b263482e DIFF:

[PATCH] D82579: [NFC] Extract unifyTargetFeatures

2020-06-25 Thread Yaxun Liu via Phabricator via cfe-commits
yaxunl updated this revision to Diff 273473. yaxunl marked 2 inline comments as done. yaxunl added a comment. return the unified features. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82579/new/ https://reviews.llvm.org/D82579 Files: clang/lib/Driver/ToolChains/Clang.cpp

[PATCH] D82332: [Coroutines] Handle dependent promise types for final_suspend non-throw check

2020-06-25 Thread Xun Li via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGc25acec84594: [Coroutines] Handle dependent promise types for final_suspend non-throw check (authored by lxfind). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D82579: [NFC] Extract unifyTargetFeatures

2020-06-25 Thread Yaxun Liu via Phabricator via cfe-commits
yaxunl marked 4 inline comments as done. yaxunl added inline comments. Comment at: clang/lib/Driver/ToolChains/CommonArgs.cpp:93 +StringRef Name = Features[I]; +assert(Name[0] == '-' || Name[0] == '+'); +LastOpt[Name.drop_front(1)] = I; tra wrote: >

[PATCH] D82477: [lldb-vscode] Add Support for Module Event

2020-06-25 Thread Greg Clayton via Phabricator via cfe-commits
clayborg added a comment. Here is a makefile that does stripping: llvm-project/lldb/test/API/lang/objc/objc-ivar-stripped/Makefile Then when creating the target, use a.out.stripped: exe = self.getBuildArtifact("a.out.stripped") symbols = exe = self.getBuildArtifact("a.out") target =

[PATCH] D81678: Introduce noundef attribute at call sites for stricter poison analysis

2020-06-25 Thread Gui Andrade via Phabricator via cfe-commits
guiand updated this revision to Diff 273456. guiand retitled this revision from "Introduce frozen attribute at call sites for stricter poison analysis" to "Introduce noundef attribute at call sites for stricter poison analysis". guiand edited the summary of this revision. guiand added a comment.

[PATCH] D81678: Introduce noundef attribute at call sites for stricter poison analysis

2020-06-25 Thread Gui Andrade via Phabricator via cfe-commits
guiand marked an inline comment as done. guiand added inline comments. Comment at: clang/lib/CodeGen/CGCall.cpp:2082 + const Type *RetTyPtr = RetTy.getTypePtr(); + if (!RetTy->isVoidType() && !RetTyPtr->isRecordType() && + RetAI.getKind() != ABIArgInfo::Indirect) {

[clang] 01c18f9 - Revert "[X86] Don't imply -mprfchw when -m3dnow is specified. Enable prefetchw in the backend with 3dnow feature."

2020-06-25 Thread Craig Topper via cfe-commits
Author: Craig Topper Date: 2020-06-25T11:43:02-07:00 New Revision: 01c18f9199ace4cb15a7c82a4163c9ef84972342 URL: https://github.com/llvm/llvm-project/commit/01c18f9199ace4cb15a7c82a4163c9ef84972342 DIFF: https://github.com/llvm/llvm-project/commit/01c18f9199ace4cb15a7c82a4163c9ef84972342.diff

[clang] c25acec - [Coroutines] Handle dependent promise types for final_suspend non-throw check

2020-06-25 Thread Xun Li via cfe-commits
Author: Xun Li Date: 2020-06-25T11:27:27-07:00 New Revision: c25acec84594ca15748553341969f8e579290e27 URL: https://github.com/llvm/llvm-project/commit/c25acec84594ca15748553341969f8e579290e27 DIFF: https://github.com/llvm/llvm-project/commit/c25acec84594ca15748553341969f8e579290e27.diff LOG:

[clang] 636d31a - [X86] Don't imply -mprfchw when -m3dnow is specified. Enable prefetchw in the backend with 3dnow feature.

2020-06-25 Thread Craig Topper via cfe-commits
Author: Craig Topper Date: 2020-06-25T11:25:35-07:00 New Revision: 636d31a5c341ff2ca5eefd6075ff059eb60b5a80 URL: https://github.com/llvm/llvm-project/commit/636d31a5c341ff2ca5eefd6075ff059eb60b5a80 DIFF: https://github.com/llvm/llvm-project/commit/636d31a5c341ff2ca5eefd6075ff059eb60b5a80.diff

[PATCH] D82415: [Coroutines] Special handle __builtin_coro_resume for final_suspend nothrow check

2020-06-25 Thread Xun Li via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. lxfind marked an inline comment as done. Closed by commit rG366159566df3: [Coroutines] Special handle __builtin_coro_resume for final_suspend nothrow… (authored by lxfind). Changed prior to commit:

[PATCH] D82306: [Sema][CodeComplete][ObjC] Don't split the first selector fragment

2020-06-25 Thread David Goldman via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGc61ef1f25c7f: [Sema][CodeComplete][ObjC] Dont split the first selector fragment (authored by dgoldman). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

  1   2   3   >