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());
--
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
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
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
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
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
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
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
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/
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
101 - 110 of 110 matches
Mail list logo