[PATCH] D50360: [Concepts] Requires Expressions

2019-10-14 Thread Saar Raz via Phabricator via cfe-commits
saar.raz added inline comments. Comment at: lib/AST/StmtProfile.cpp:1325 + //expression. It is equivalent to the simple-requirement x++; [...] + // We therefore do not profile isSimple() here. + ID.AddBoolean(ExprReq->getNoexceptLoc().isValid()); --

[PATCH] D62686: [RISCV] Add support for save/restore of callee-saved registers via libcalls

2019-10-14 Thread Ana Pazos via Phabricator via cfe-commits
apazos added a comment. Yes Eli thanks for pointing out there are more scenarios that can fail. It looks like the best solution is to permit both flags on, but then bail out from doing this transformation if there is any type of tail call already in the function. This way we avoid messing with

Re: [llvm-dev] Attention bot owners

2019-10-14 Thread Galina Kistanova via cfe-commits
Hi Diana, git 2.7.4 should be fine. Thanks Galina On Mon, Oct 14, 2019 at 12:57 AM Diana Picus wrote: > Hi Galina, > > Thanks for the heads up. > > On Fri, 11 Oct 2019 at 21:09, Galina Kistanova via llvm-dev > wrote: > > > > Hello all bots owners, > > > > As all of you know we move to githu

LLVM buildmaster will be updated tonight

2019-10-14 Thread Galina Kistanova via cfe-commits
Hello everyone, LLVM buildmaster will be updated and restarted after 7PM Pacific time today. Thank Galina ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r374856 - [WebAssembly] Trapping fptoint builtins and intrinsics

2019-10-14 Thread Thomas Lively via cfe-commits
Author: tlively Date: Mon Oct 14 18:11:51 2019 New Revision: 374856 URL: http://llvm.org/viewvc/llvm-project?rev=374856&view=rev Log: [WebAssembly] Trapping fptoint builtins and intrinsics Summary: The WebAssembly backend lowers fptoint instructions to a code sequence that checks for overflow to

[PATCH] D68902: [WebAssembly] Trapping fptoint builtins and intrinsics

2019-10-14 Thread Thomas Lively via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG232fd99d9ee6: [WebAssembly] Trapping fptoint builtins and intrinsics (authored by tlively). Changed prior to commit: https://reviews.llvm.org/D68902?vs=224718&id=224945#toc Repository: rG LLVM Github

Zorg migration to GitHub/monorepo

2019-10-14 Thread Galina Kistanova via cfe-commits
Hello everyone, We are in the middle of porting the majority of zorg to GitHub/monorepo. The following build factories will be ported and if you use one of those for your bots, you are all covered: * ClangBuilder.getClangCMakeBuildFactory (31 bots) * ClangBuilder.getClangCMakeGCSBuildFactory

[PATCH] D68969: [clang-format] Remove the dependency on frontend

2019-10-14 Thread MyDeveloperDay via Phabricator via cfe-commits
MyDeveloperDay created this revision. MyDeveloperDay added reviewers: thakis, klimek, mitchell-stellar. MyDeveloperDay added projects: clang, clang-format. Herald added a subscriber: mgorny. Address review comments from D68554: [clang-format] Proposal for clang-format to give compiler style warni

[PATCH] D68927: [clang-tools-extra] Fix overzealous linking of dylib to clangTidy

2019-10-14 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru accepted this revision. sylvestre.ledru added a comment. This revision is now accepted and ready to land. Thanks It fixed my problem. Maybe you could improve the doc and/or comment to make this behavior more explicit? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68927/

[PATCH] D63640: [clang] Improve Serialization/Imporing/Dumping of APValues

2019-10-14 Thread Tyker via Phabricator via cfe-commits
Tyker added a comment. ping @rsmith CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63640/new/ https://reviews.llvm.org/D63640 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

<    1   2