dnovillo added a comment.
In https://reviews.llvm.org/D23284#526389, @anemet wrote:
> @dnovillo or @rsmith, can you please confirm if you agree that this new
> option -Rpass-with-hotness should not be part of R_group. R_group options
> enable/disable groups of remarks whereas this one is only
On Fri, Oct 2, 2015 at 9:44 AM, Teresa Johnson via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
As David mentioned, "inlineonly" is much too restrictive for what is
> possible. I prefer to stick with "thin" since it refers to this new model
> of keeping the whole program part very thin.
>
Agr
On Mon, Oct 5, 2015 at 9:55 AM, Benjamin Kramer via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: d0k
> Date: Mon Oct 5 08:55:20 2015
> New Revision: 249316
>
> URL: http://llvm.org/viewvc/llvm-project?rev=249316&view=rev
> Log:
> [VFS] Add working directories to every virtual file s
Never mind. My llvm and clang trees were out of sync.
Diego.
On Mon, Oct 5, 2015 at 11:15 AM, Diego Novillo wrote:
>
>
> On Mon, Oct 5, 2015 at 9:55 AM, Benjamin Kramer via cfe-commits <
> cfe-commits@lists.llvm.org> wrote:
>
>> Author: d0k
>> Date: Mon Oct 5 08:55:20 2015
>> New Revision: 2
Author: dnovillo
Date: Tue Aug 25 10:25:13 2015
New Revision: 245941
URL: http://llvm.org/viewvc/llvm-project?rev=245941&view=rev
Log:
Convert SampleProfile pass into a Module pass.
Eventually, we will need sample profiles to be incorporated into the
inliner's cost models. To do this, we need th
Author: dnovillo
Date: Wed Oct 14 13:37:39 2015
New Revision: 250310
URL: http://llvm.org/viewvc/llvm-project?rev=250310&view=rev
Log:
Sample profiles - Update text profile documentation.
There's been some changes to the text encoding for sample profiles. This
updates the documentation and an exa
Author: dnovillo
Date: Mon Oct 19 10:53:17 2015
New Revision: 250705
URL: http://llvm.org/viewvc/llvm-project?rev=250705&view=rev
Log:
Sample Profiles - Fix location of binary encoding documentation. NFC.
Modified:
cfe/trunk/docs/UsersManual.rst
Modified: cfe/trunk/docs/UsersManual.rst
URL:
dnovillo accepted this revision.
dnovillo added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rL LLVM
http://reviews.llvm.org/D21737
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-
On Sat, Jul 9, 2016 at 7:39 PM Sean Silva wrote:
> silvas added a comment.
>
> In http://reviews.llvm.org/D21823#479418, @davidxl wrote:
>
> > I should have brought it up earlier, but I forgot.I think a better
> (and simpler) proposal is to make -fprofile-generate and -fprofile-use turn
> on
Nico, this is producing tons of warnings on an LLVM build and is actually
breaking our internal builds (we build with -Werror).
I fixed one file that was producing this, but there are several that have
the same problem (e.g., gtest-port.h). Could you fix them or rollback?
Thanks. Diego.
On Tu
On Tue, Jan 19, 2016 at 3:30 PM, Nico Weber wrote:
> I'll take a look. If it's urgent it's also possible to disable this
> warning.
>
Yeah. I think disabling it by default may be the better choice. Thanks.
___
cfe-commits mailing list
cfe-commits@lis
On Tue, Jan 19, 2016 at 3:43 PM, Nico Weber wrote:
> I mean you could pass a -Wno flag. It's undefined behavior that's actually
> causing bugs in practice; it should probably be on by default.
>
But then you need to fix all the warnings it's producing in an llvm build.
That is currently blocking
Thanks!
On Jan 19, 2016 3:34 PM, "Nico Weber" wrote:
> I think I've fixed all warnings by now, and I've reenabled the warning for
> LLVM builds (in r258210; that commit's commit message links to the
> individual fixes.)
>
> On Tue, Jan 19, 2016 at 3:56 PM, Nico Weber wrote:
>
>> r258181 should s
Author: dnovillo
Date: Wed Dec 16 13:52:05 2015
New Revision: 255809
URL: http://llvm.org/viewvc/llvm-project?rev=255809&view=rev
Log:
Add -fsyntax-only to fix failure in read-only directories.
Internally, this test is executed in a read-only directory, which causes
it to fail because the driver
14 matches
Mail list logo
