LGTM
On Thu, May 24, 2012 at 10:27 AM, Manuel Klimek kli...@google.com wrote:
Proposal to link the tooling docs into the main page.
Cheers,
/Manuel
Index: menu.html.incl
===
--- menu.html.incl (revision 157394)
+++
Author: klimek
Date: Fri May 25 03:39:21 2012
New Revision: 157462
URL: http://llvm.org/viewvc/llvm-project?rev=157462view=rev
Log:
Links the entry point to the tooling documentation into clang's menu bar.
Modified:
cfe/trunk/www/menu.html.incl
Modified: cfe/trunk/www/menu.html.incl
URL:
On Wed, May 23, 2012 at 4:07 PM, Evgeniy Stepanov
eugeni.stepa...@gmail.com wrote:
Good point. Both patches landed, here is another one that adds a
driver check for -pie.
ping
On Wed, May 23, 2012 at 3:04 PM, Chandler Carruth chandl...@google.com
wrote:
On Wed, May 23, 2012 at 3:58 AM,
On Wed, May 23, 2012 at 4:07 PM, Evgeniy Stepanov
eugeni.stepa...@gmail.com wrote:
Good point. Both patches landed, here is another one that adds a
driver check for -pie.
ping w/ a patch
On Wed, May 23, 2012 at 3:04 PM, Chandler Carruth chandl...@google.com
wrote:
On Wed, May 23, 2012 at
On Fri, May 25, 2012 at 3:16 AM, Evgeniy Stepanov eugeni.stepa...@gmail.com
wrote:
On Wed, May 23, 2012 at 4:07 PM, Evgeniy Stepanov
eugeni.stepa...@gmail.com wrote:
Good point. Both patches landed, here is another one that adds a
driver check for -pie.
ping w/ a patch
LGTM
The
On Thu, May 24, 2012 at 05:50:00PM -0700, Marshall Clow wrote:
On May 24, 2012, at 5:03 PM, Seth Cantrell wrote:
Author: socantre
Date: Thu May 24 19:03:29 2012
New Revision: 157445
URL: http://llvm.org/viewvc/llvm-project?rev=157445view=rev
Log:
ensure value passed to is space
On May 25, 2012, at 3:26 AM, Joerg Sonnenberger wrote:
On Thu, May 24, 2012 at 05:50:00PM -0700, Marshall Clow wrote:
On May 24, 2012, at 5:03 PM, Seth Cantrell wrote:
Author: socantre
Date: Thu May 24 19:03:29 2012
New Revision: 157445
URL:
Author: rafael
Date: Fri May 25 09:17:45 2012
New Revision: 157469
URL: http://llvm.org/viewvc/llvm-project?rev=157469view=rev
Log:
Whitespace fixes.
Modified:
cfe/trunk/lib/AST/Decl.cpp
Modified: cfe/trunk/lib/AST/Decl.cpp
URL:
Author: rafael
Date: Fri May 25 09:47:05 2012
New Revision: 157470
URL: http://llvm.org/viewvc/llvm-project?rev=157470view=rev
Log:
When ignoring visibility in an instantiation, still consider the linkage.
Similar fixes for function and member template to follow as I write the
testcases.
Author: rafael
Date: Fri May 25 10:51:26 2012
New Revision: 157475
URL: http://llvm.org/viewvc/llvm-project?rev=157475view=rev
Log:
Consider the linkage for member class templates even when we have to ignore
the visibility.
Modified:
cfe/trunk/lib/AST/Decl.cpp
Author: hhinnant
Date: Fri May 25 10:55:46 2012
New Revision: 157476
URL: http://llvm.org/viewvc/llvm-project?rev=157476view=rev
Log:
Fix memory corruption bug found and fixed by Andrew C. Morrow.
Modified:
libcxx/trunk/include/future
Modified: libcxx/trunk/include/future
URL:
Author: zaks
Date: Fri May 25 11:02:16 2012
New Revision: 157478
URL: http://llvm.org/viewvc/llvm-project?rev=157478view=rev
Log:
[analyzer] Don't crash on LValBitCast
Modified:
cfe/trunk/lib/StaticAnalyzer/Core/ExprEngineC.cpp
cfe/trunk/test/Analysis/cxx11-crashes.cpp
Modified:
I recently needed to write code to iterate over preprocessing line
control directives, identify the originating source file for the
directive, and then lookup details for the originating source file.
LineTableInfo has begin() and end() methods to iterate over the set of
source file IDs and
Do we even need to conjure a new value here? reinterpret_cast on a location
should be the same as static_cast or C-style casts, right? (At least when we're
not dealing with fptr-ptr conversions or different memory segments.)
Of course, I might be missing something.
On May 25, 2012, at 12:02,
Author: rafael
Date: Fri May 25 11:41:35 2012
New Revision: 157480
URL: http://llvm.org/viewvc/llvm-project?rev=157480view=rev
Log:
Don't ignore linkage when ignoring visibility in the instantiation of a
function template.
Modified:
cfe/trunk/lib/AST/Decl.cpp
Author: nlopes
Date: Fri May 25 12:04:42 2012
New Revision: 157483
URL: http://llvm.org/viewvc/llvm-project?rev=157483view=rev
Log:
add CodeGen support for the alloc_size attribute
Added:
cfe/trunk/test/CodeGen/alloc_size.c
Modified:
cfe/trunk/lib/CodeGen/CGCall.cpp
Modified:
Author: rafael
Date: Fri May 25 12:22:33 2012
New Revision: 157486
URL: http://llvm.org/viewvc/llvm-project?rev=157486view=rev
Log:
Don't ignore linkage when ignoring visibility in the instantiation of a
method template.
Modified:
cfe/trunk/lib/AST/Decl.cpp
Hi cfe-commits,
This patch handles a specific (but surprisingly common) case when a
fall-through occurs to a switch label immediately followed by a break;. In
this case it doesn't make sense to suggest a fall-through annotation,
almost certainly inserting break; is a good fix-it.
Example:
switch
All good now. Thanks!
On Thu, May 24, 2012 at 2:07 PM, Eli Friedman eli.fried...@gmail.com wrote:
On Thu, May 24, 2012 at 9:57 AM, Nico Weber tha...@chromium.org wrote:
Hi Eli,
someone on IRC reported that this breaks the compiler-rt build. Log at
http://paste.ubuntu.com/1004855/ , excerpt
Author: akirtzidis
Date: Fri May 25 15:05:57 2012
New Revision: 157491
URL: http://llvm.org/viewvc/llvm-project?rev=157491view=rev
Log:
[libclang] Add a test I forgot to commit for r156890.
Added:
cfe/trunk/test/Index/index-refs.m
Added: cfe/trunk/test/Index/index-refs.m
URL:
Suggestion: use the stack-based SmallVector instead of std::vector. Also, a
CXXMethodDecl might be a static method, so you'll need to check isInstance() as
well.
Jordy
On May 25, 2012, at 13:04, Nuno Lopes wrote:
Author: nlopes
Date: Fri May 25 12:04:42 2012
New Revision: 157483
URL:
On Fri, May 25, 2012 at 11:15 AM, Alexander Kornienko ale...@google.comwrote:
Hi cfe-commits,
This patch handles a specific (but surprisingly common) case when a
fall-through occurs to a switch label immediately followed by a break;.
In this case it doesn't make sense to suggest a
Nuno, this patch didn't get reviewed before commit. The LLVM review is
ongoing, and seems to indicate the solution is not yet ready. Folks have
also raised serious concerns about the implementation strategy here.
While supporting the alloc_size attribute at least enough to parse it and
reject
Author: nlopes
Date: Fri May 25 16:45:08 2012
New Revision: 157500
URL: http://llvm.org/viewvc/llvm-project?rev=157500view=rev
Log:
fix codegen support for alloc_size attribute for static C++ methods
add test case for C++ codegen
Added:
cfe/trunk/test/CodeGenCXX/alloc_size.cpp
Modified:
Fixed in r157500.
Thanks for the review,
Nuno
Citando Jordy Rose jedik...@belkadan.com:
Suggestion: use the stack-based SmallVector instead of std::vector.
Also, a CXXMethodDecl might be a static method, so you'll need to
check isInstance() as well.
Jordy
On May 25, 2012, at 13:04,
Author: hhinnant
Date: Fri May 25 17:04:21 2012
New Revision: 157503
URL: http://llvm.org/viewvc/llvm-project?rev=157503view=rev
Log:
The rules for emplace in map, multimap, unordered_map and unordered_multimap
changed a while back and I'm just now updating to these new rules. In a
nutshell,
There has been some discussion on both the bounds checking and the
alloc_size proposals on the various mailing lists (dev commits) for
the past month.
Yes, the design is not finished. But I've been making changes to the
implementation to accommodate the feedback.
Nuno
Citando Chandler
On May 25, 2012, at 3:23 PM, Nuno Lopes nunoplo...@sapo.pt wrote:
There has been some discussion on both the bounds checking and the alloc_size
proposals on the various mailing lists (dev commits) for the past month.
Yes, the design is not finished. But I've been making changes to the
The primary concerns I have seen so far are it doesn't serve the needs for all
memory safety techniques. Did I miss some other specific concern about
alloc_size attribute?
I think I should address the recent discussions the LLVM side wrt to the bounds
checking pass. I've been following the
Hi cfe-commits,
This patch adds a soft opt-in option for
-Wimplicit-fallthroughdiagnostics. The reason for it is to provide a
way to start using
fall-through annotations without breaking (when treating warnings as
errors) all code with unannotated fall-throughs. So it's only meant to be
used for
Hi Chandler,
On Fri, May 25, 2012 at 11:15 PM, Chandler Carruth chandl...@google.comwrote:
On Fri, May 25, 2012 at 11:15 AM, Alexander Kornienko
ale...@google.comwrote:
Hi cfe-commits,
This patch handles a specific (but surprisingly common) case when a
fall-through occurs to a switch
Author: alexfh
Date: Fri May 25 19:49:15 2012
New Revision: 157508
URL: http://llvm.org/viewvc/llvm-project?rev=157508view=rev
Log:
Don't offer '[[clang::fallthrough]];' fix-it when a fall-through occurs to a
switch label immediately followed by a 'break;'.
Modified:
If it looks good, could you commit it? I don't have commit access yet.
No worries, I got commit access and committed it as r157508.
--
Regards,
Alex
___
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
On May 25, 2012, at 3:43 PM, Evan Cheng wrote:
The primary concerns I have seen so far are it doesn't serve the needs for
all memory safety techniques. Did I miss some other specific concern about
alloc_size attribute?
On another note, one part of Nuno's goal for the summer is to implement
On May 24, 2012, at 10:03 PM, Charles Davis wrote:
On May 24, 2012, at 5:53 AM, John McCall wrote:
On May 24, 2012, at 4:35 AM, Timur Iskhodzhanov wrote:
On Thu, May 24, 2012 at 2:44 PM, João Matos ripzonetri...@gmail.com wrote:
Attached is a patch that introduces template mangling (at least
On May 24, 2012, at 3:17 AM, Erik Verbruggen eri...@me.com wrote:
Quite errr... surprising. I only understood the testcase after reading it
like 3 times. I think we should add a warning for this (possibly off by
default), because I can see a lot of people falling into this trap.
I'd be
Author: dblaikie
Date: Sat May 26 00:35:39 2012
New Revision: 157510
URL: http://llvm.org/viewvc/llvm-project?rev=157510view=rev
Log:
Fix indentation.
Modified:
cfe/trunk/lib/Sema/SemaDecl.cpp
Modified: cfe/trunk/lib/Sema/SemaDecl.cpp
URL:
37 matches
Mail list logo