On 10/30/2018 2:58 PM, Roman Lebedev via cfe-commits wrote:
Author: lebedevri
Date: Tue Oct 30 14:58:56 2018
New Revision: 345660
URL: http://llvm.org/viewvc/llvm-project?rev=345660&view=rev
Log:
[clang][ubsan] Implicit Conversion Sanitizer - integer sign change - clang part
This is the second
On 10/10/2018 4:13 PM, Richard Smith via cfe-commits wrote:
Author: rsmith
Date: Wed Oct 10 16:13:35 2018
New Revision: 344199
URL: http://llvm.org/viewvc/llvm-project?rev=344199&view=rev
Log:
Add a flag to remap manglings when reading profile data information.
This can be used to preserve prof
On 7/30/2018 3:44 AM, Stefan Maksimovic via cfe-commits wrote:
Author: smaksimovic
Date: Mon Jul 30 03:44:46 2018
New Revision: 338239
URL: http://llvm.org/viewvc/llvm-project?rev=338239&view=rev
Log:
[mips64][clang] Provide the signext attribute for i32 return values
Additional info: see r3380
On 5/2/2018 9:42 AM, Jessica Paquette via cfe-commits wrote:
Author: paquette
Date: Wed May 2 09:42:51 2018
New Revision: 331370
URL: http://llvm.org/viewvc/llvm-project?rev=331370&view=rev
Log:
Add -foutline option to enable the MachineOutliner in AArch64
It probably makes sense to also have
The test wasn't checking anything useful; the clang driver never passes
"-arm-long-calls" to clang -cc1. See https://reviews.llvm.org/rL241565 .
-Eli
On 4/12/2018 3:55 PM, George Karpenkov wrote:
Hi Eli,
The commit makes sense, but I’m not sure about your change to the
`test/Driver/apple-ke
On 10/25/2017 12:49 PM, Reid Kleckner wrote:
On Wed, Oct 25, 2017 at 12:14 PM, Friedman, Eli
mailto:efrie...@codeaurora.org>> wrote:
Maybe we should just ignore -fno-short-wchar, instead? I think
that's what gcc and released versions of clang do (that means
-fno-short-wchar doesn't
On 10/25/2017 11:54 AM, Reid Kleckner wrote:
On Wed, Oct 25, 2017 at 10:56 AM, Friedman, Eli
mailto:efrie...@codeaurora.org>> wrote:
On 10/6/2017 4:09 PM, Saleem Abdulrasool via cfe-commits wrote:
Modified: cfe/trunk/lib/Driver/ToolChains/Clang.cpp
URL:
http://llvm
On 10/6/2017 4:09 PM, Saleem Abdulrasool via cfe-commits wrote:
Modified: cfe/trunk/lib/Driver/ToolChains/Clang.cpp
URL:
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains/Clang.cpp?rev=315126&r1=315125&r2=315126&view=diff
On 10/17/2017 5:00 PM, Saleem Abdulrasool via cfe-commits wrote:
Author: compnerd
Date: Tue Oct 17 17:00:50 2017
New Revision: 316046
URL: http://llvm.org/viewvc/llvm-project?rev=316046&view=rev
Log:
Basic: fix __{,U}INTPTR_TYPE__ on ARM
Darwin and OpenBSD are the only platforms which use `long
Sometimes people just lose track; "pinging" is normal (see
http://llvm.org/docs/DeveloperPolicy.html#code-reviews). And sometimes
it's helpful to CC reviewers in the area; adding Alexander Kornienko.
-Eli
On 10/17/2017 12:03 PM, Antoni Boucher wrote:
Since the patch was redirected to the rig
On 10/4/2017 1:48 PM, Keane, Erich wrote:
Ah, cool! I didn’t realize that, and that is actually exactly what
Elizabeth is looking into now.
Elizabeth, if you send me a diff, I’ll commit it as
review-after-commit if Eli is alright with this. It should be
something like changing:
+ i
Oh, that's easy to explain; sorry, I didn't think of it when I was
reviewing.
DefinitionKind has three possible values: DeclarationOnly,
TentativeDefinition, and Definition. (Tentative definitions are C
weirdness that allows you to write "int x; int x = 10;".)
For the purpose of this warnin
When you're submitting a patch, please make sure you're sending it to
the right list, and the "subject" line actually describes what the patch
changes; otherwise, reviewers won't notice the patch. Optionally, you
can submit a patch using Phabricator, a tool which which makes patch
reviews a li
On 9/26/2017 11:55 AM, Erich Keane via cfe-commits wrote:
Author: erichkeane
Date: Tue Sep 26 11:55:16 2017
New Revision: 314235
URL: http://llvm.org/viewvc/llvm-project?rev=314235&view=rev
Log:
Allow IUnknown/IInterface types to come from extern C++
It was brought up in response to my last imp
On 9/21/2017 12:58 PM, Erich Keane via cfe-commits wrote:
Author: erichkeane
Date: Thu Sep 21 12:58:55 2017
New Revision: 313907
URL: http://llvm.org/viewvc/llvm-project?rev=313907&view=rev
Log:
Suppress Wsign-conversion for enums with matching underlying type
As reported here: https://bugs.llv
On 9/8/2017 2:18 PM, Hans Wennborg via cfe-commits wrote:
On Fri, Sep 8, 2017 at 2:09 PM, Roman Lebedev wrote:
Interesting. My first thought was to explicitly specify enum as signed:
enum MediaDeviceType : signed int {
MEDIA_DEVICE_TYPE_AUDIO_INPUT = 0,
MEDIA_DEVICE_TYPE_VIDEO_INPUT,
MEDIA_DE
On 5/5/2017 10:20 AM, Nico Weber via cfe-commits wrote:
Timestamps on cfe-commits are a bit messed up. I landed that just now,
but the timestamp is from 13 minutes in the past. Maybe the clock on
the mail server is off by 13 min?
The timestamp on the message matches the timestamp in the svn lo
On 1/6/2017 3:05 PM, Andrew Rogers wrote:
Hi,
This is a (tested!) patch to add checks that both pre-increment and
pre-decrement don’t overflow, for both char and short integer types
(since they’re both narrower than int), as per the problem fixed in
PR9350.
r291805.
-Eli
--
Employee o
On 12/20/2016 6:56 PM, Richard Smith wrote:
On 20 December 2016 at 18:14, Richard Smith <mailto:rich...@metafoo.co.uk>> wrote:
On 19 December 2016 at 17:00, Friedman, Eli via cfe-commits
mailto:cfe-commits@lists.llvm.org>>
wrote:
On 12/19/2016 3:59 PM, Richar
On 12/19/2016 3:59 PM, Richard Smith via cfe-commits wrote:
Author: rsmith
Date: Mon Dec 19 17:59:34 2016
New Revision: 290146
URL:
http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Sema/vfprintf-valid-redecl.c?rev=290146&r1=290145&r2=290146&view=diff
==
It probably makes sense to say that alloca should have the same
alignment as the default stack alignment, simply because it's hard to do
anything useful with completely unaligned memory. That said, the
default stack alignment is 4 bytes on 32-bit Windows, not 16.
-Eli
On 9/9/2016 10:29 AM, R
21 matches
Mail list logo
