lebedev.ri marked an inline comment as done.
lebedev.ri added a comment.
Bump @rsmith
thanks
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D73020/new/
https://reviews.llvm.org/D73020
___
cfe-commits mai
lebedev.ri added a comment.
Bump @rsmith / @erichkeane
thanks
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D73996/new/
https://reviews.llvm.org/D73996
___
cfe-commits mailing list
cfe-commits@lists.ll
Author: Michael Liao
Date: 2020-02-10T00:15:56-05:00
New Revision: a06789138987d1f64bb2f97d3a5c0f39eaf94715
URL:
https://github.com/llvm/llvm-project/commit/a06789138987d1f64bb2f97d3a5c0f39eaf94715
DIFF:
https://github.com/llvm/llvm-project/commit/a06789138987d1f64bb2f97d3a5c0f39eaf94715.diff
davezarzycki added a comment.
Hi @MaskRay – I was just looking at the source history. I didn't test the old
way. As it turns out there isn't a regression, but just some unexplained code
in the current clang option parsing. And the `-disable-O0-optnone` is a useful
hint. Thanks! I've been hackin
Author: Jan Vesely
Date: 2020-02-09T14:42:15-05:00
New Revision: 85e2fa44c64e1edd2f675c990ecc60f5fadb4686
URL:
https://github.com/llvm/llvm-project/commit/85e2fa44c64e1edd2f675c990ecc60f5fadb4686
DIFF:
https://github.com/llvm/llvm-project/commit/85e2fa44c64e1edd2f675c990ecc60f5fadb4686.diff
LO
Author: Jan Vesely
Date: 2020-02-09T14:42:09-05:00
New Revision: 4b23a2e8e971876d075d3ae322754dbc0495413d
URL:
https://github.com/llvm/llvm-project/commit/4b23a2e8e971876d075d3ae322754dbc0495413d
DIFF:
https://github.com/llvm/llvm-project/commit/4b23a2e8e971876d075d3ae322754dbc0495413d.diff
LO
On Sun, 9 Feb 2020, 01:09 Nico Weber via cfe-commits, <
cfe-commits@lists.llvm.org> wrote:
> Our code fails to build with "destructor cannot be declared using a type
> alias" after this, without us changing language mode or anything.
>
> Is that intended?
>
Can you provide a sketch of what you we
MaskRay added a comment.
Why you say regression, do you have a case where
fcd33149b48885ab8e4ca4ffb6977bce5be2e623 actually regressed things?
-O0 implies optnone. For `-Xclang -disable-O0-optnone -O0 -finline-functions`,
the patch can indeed make a difference. `-disable-O0-optnone` is currently
davezarzycki added a comment.
Hi @lebedev.ri – Before I add a small new test file, is there an existing one
that you know of that would be good to extend?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D74298/new/
https://reviews.llvm.org/D74298
lebedev.ri added a comment.
This clearly needs tests
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D74298/new/
https://reviews.llvm.org/D74298
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
htt
davezarzycki created this revision.
davezarzycki added a reviewer: chandlerc.
davezarzycki added a project: clang.
This fixes a regression introduced (over three years ago!) by
fcd33149b48885ab8e4ca4ffb6977bce5be2e623.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D74298
File
llunak added a comment.
Ping again. Is there still something more to do here?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69585/new/
https://reviews.llvm.org/D69585
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llv
Author: serge_sans_paille
Date: 2020-02-09T10:42:45+01:00
New Revision: e67cbac81211d40332a79d98c9d5953624cc1202
URL:
https://github.com/llvm/llvm-project/commit/e67cbac81211d40332a79d98c9d5953624cc1202
DIFF:
https://github.com/llvm/llvm-project/commit/e67cbac81211d40332a79d98c9d5953624cc1202.d
Author: serge-sans-paille
Date: 2020-02-09T10:06:31+01:00
New Revision: 454621160066c067c97edb0a094553d8d0339c9b
URL:
https://github.com/llvm/llvm-project/commit/454621160066c067c97edb0a094553d8d0339c9b
DIFF:
https://github.com/llvm/llvm-project/commit/454621160066c067c97edb0a094553d8d0339c9b.d
AlexanderLanin added a comment.
On second thought maybe this should be fixed in clang-tidy and not here?
Maybe besides `-export-fixes` there should be an `-export-warnings` or just
`-yaml-export`?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72730/new/
https://reviews.llvm.org/D72730
Author: serge_sans_paille
Date: 2020-02-09T09:35:42+01:00
New Revision: 0fd51a4554f5f4f90342f40afd35b077f6d88213
URL:
https://github.com/llvm/llvm-project/commit/0fd51a4554f5f4f90342f40afd35b077f6d88213
DIFF:
https://github.com/llvm/llvm-project/commit/0fd51a4554f5f4f90342f40afd35b077f6d88213.d
16 matches
Mail list logo