On Dec 12, 2012, at 11:45 PM, endlessroad1...@gmail.com wrote:
Thanks, that's a very detailed and thorough comment.
MSDN page for __declspec(property):
http://msdn.microsoft.com/en-us/library/yhfk0thd.aspx
we can declare array-like property:
__declspec(property(get=GetX,
Il 13/12/2012 02:17, João Matos ha scritto:
On Thu, Dec 13, 2012 at 12:48 AM, Will Wilson w...@indefiant.com
mailto:w...@indefiant.com wrote:
Thanks for the info Joåo. Do you think it'll be a lot of work to get
to get it working against the test case?
It does not seem too hard
http://llvm-reviews.chandlerc.com/D214
Files:
lib/AST/ASTDumper.cpp
lib/AST/StmtDumper.cpp
lib/AST/CMakeLists.txt
Index: lib/AST/ASTDumper.cpp
===
--- lib/AST/ASTDumper.cpp
+++ lib/AST/ASTDumper.cpp
@@ -1,13 +1,13 @@
-//===---
On Thu, Dec 13, 2012 at 5:06 AM, Jordan Rose jordan_r...@apple.com wrote:
+/// \brief Determine whether the given class has a superclass that we want
+/// to check. The name of the found superclass is stored in SuperclassName.
+///
+/// \param ObjCImplementationDecl The declaration to check
Author: gbenyei
Date: Thu Dec 13 07:22:48 2012
New Revision: 170125
URL: http://llvm.org/viewvc/llvm-project?rev=170125view=rev
Log:
Remove little endian specification from SPIR data layout - SPIR doesn't define
endiannes in the data layout.
Modified:
cfe/trunk/lib/Basic/Targets.cpp
Looks good.
http://llvm-reviews.chandlerc.com/D214
___
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
Author: alexfh
Date: Thu Dec 13 07:59:55 2012
New Revision: 170127
URL: http://llvm.org/viewvc/llvm-project?rev=170127view=rev
Log:
Rename StmtDumper to ASTDumper.
Patch contributed by Philip Craig!
Added:
cfe/trunk/lib/AST/ASTDumper.cpp
- copied, changed from r170126,
Awesome!
On 13 December 2012 02:11, Richard Smith richard-l...@metafoo.co.uk wrote:
Author: rsmith
Date: Thu Dec 13 01:11:50 2012
New Revision: 170108
URL: http://llvm.org/viewvc/llvm-project?rev=170108view=rev
Log:
ubsan: Add -fsanitize=bool and -fsanitize=enum, which check for loads of
Hello,
A minor change to the documentation of getBase() in Expr.h for MemberExpr to
indicate what getBase() does.
--
Hiren
Expr.h.patch
Description: Binary data
___
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
Looks good.
http://llvm-reviews.chandlerc.com/D211
BRANCH
stylefix
ARCANIST PROJECT
clang
___
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
Author: gribozavr
Date: Thu Dec 13 10:06:00 2012
New Revision: 170133
URL: http://llvm.org/viewvc/llvm-project?rev=170133view=rev
Log:
Documentation: add AutomaticReferenceCounting.rst to the toctree
Modified:
cfe/trunk/docs/index.rst
Modified: cfe/trunk/docs/index.rst
URL:
Author: rdivacky
Date: Thu Dec 13 10:09:42 2012
New Revision: 170135
URL: http://llvm.org/viewvc/llvm-project?rev=170135view=rev
Log:
Dont use/link ARCMT, StaticAnalyzer and Rewriter to clang when the user
specifies not to. Dont build ASTMatchers with Rewriter disabled and
StaticAnalyzer when
Author: chapuni
Date: Thu Dec 13 10:24:59 2012
New Revision: 170136
URL: http://llvm.org/viewvc/llvm-project?rev=170136view=rev
Log:
Update CMake build corresponding to r170135.
Thanks to Saleem Abdulrasool, aka compnerd!
Modified:
cfe/trunk/CMakeLists.txt
Modified:
Author: pawel
Date: Thu Dec 13 10:31:31 2012
New Revision: 170137
URL: http://llvm.org/viewvc/llvm-project?rev=170137view=rev
Log:
Merging r170026: into the 3.2 release branch.
Zhang Xiongpang: Add definitions for const data members. Fixes
http://llvm.org/bugs/show_bug.cgi?id=14585.
Added:
Hi, Anton. Thanks for looking at this.
The reason CXXNewExpr does not currently have pre- and post- statement checks
is because it's not clear what a pre-statement check means. By the time the
analyzer reaches the CXXNewExpr, the constructor for the object has already
been run (because the
Ha, oops! Thanks, Dmitri. Indeed, -Wdocumentation got the correct fix there;
either Julian or I should have caught that! (And I should reset my build to add
-Wdocumentation to my warnings...)
I don't have a clean tree right now but I'll check in the fix soon.
Jordan
On Dec 13, 2012, at 5:19
A minor change to the documentation of getBase() in Expr.h for MemberExpr to
indicate what getBase() does.
Looks good to me. Please commit.
- David
___
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
LGTM. I was about to ask for a testcase, but we hard code
/etc/lsb-release, so we need some refactoring before this can be
tested :-(
Do you have commit access?
On 12 December 2012 04:26, Martin Nowack mar...@se.inf.tu-dresden.de wrote:
Hi,
Attached is a patch for current Ubuntu Quantal and
Looks good.
http://llvm-reviews.chandlerc.com/D212
BRANCH
master
ARCANIST PROJECT
clang-tools-extra
___
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
Author: jrose
Date: Thu Dec 13 12:26:05 2012
New Revision: 170139
URL: http://llvm.org/viewvc/llvm-project?rev=170139view=rev
Log:
[analyzer] Fix doc error (wrong param name) in ObjCSuperCallChecker.
Thanks for the -Wdocumentation catch, Dmitri!
Modified:
Thanks, that's a very detailed and thorough comment.
MSDN page for __declspec(property):
http://msdn.microsoft.com/en-us/library/yhfk0thd.aspx
we can declare array-like property:
__declspec(property(get=GetX, put=PutX)) int x[][];
and access it with indices:
var =
Fixed in r170139.
On Dec 13, 2012, at 9:10 , Jordan Rose jordan_r...@apple.com wrote:
Ha, oops! Thanks, Dmitri. Indeed, -Wdocumentation got the correct fix there;
either Julian or I should have caught that! (And I should reset my build to
add -Wdocumentation to my warnings...)
I don't
Per request from Rafael, I have applied these release 3.1 patches to trunk.
They applied cleanly. The resulting diff is attached.
Please review to significantly improve clang's support on DragonFly!
+ bool UseGCC47;
...
+ llvm::sys::fs::exists(/usr/lib/gcc47, UseGCC47);
UseGCC47 can be
Author: nadav
Date: Thu Dec 13 13:58:10 2012
New Revision: 170143
URL: http://llvm.org/viewvc/llvm-project?rev=170143view=rev
Log:
Revert 170049 because it fails with an assertion on one of the spec2000
workloads.
Modified:
cfe/trunk/lib/Frontend/DiagnosticRenderer.cpp
Hi Rafael,
No, I don't have commit access.
Would be great if you or somebody else could push it.
Thanks a lot.
Martin
On Dec 13, 2012, at 7:21 PM, Rafael Espíndola rafael.espind...@gmail.com
wrote:
LGTM. I was about to ask for a testcase, but we hard code
/etc/lsb-release, so we need some
Author: rafael
Date: Thu Dec 13 14:26:05 2012
New Revision: 170147
URL: http://llvm.org/viewvc/llvm-project?rev=170147view=rev
Log:
Add support for current Ubuntu Quantal and the upcoming Raring.
Patch by Martin Nowack.
Modified:
cfe/trunk/lib/Driver/ToolChains.cpp
Modified:
On 13 December 2012 15:19, Martin Nowack mar...@se.inf.tu-dresden.de wrote:
Hi Rafael,
No, I don't have commit access.
Would be great if you or somebody else could push it.
r170147. Thanks for the patch.
Thanks a lot.
Martin
Cheers,
Rafael
___
On 12/13/2012 21:02, Rafael Espíndola wrote:
Per request from Rafael, I have applied these release 3.1 patches to trunk.
They applied cleanly. The resulting diff is attached.
Please review to significantly improve clang's support on DragonFly!
+ bool UseGCC47;
...
+
Author: akirtzidis
Date: Thu Dec 13 15:38:23 2012
New Revision: 170150
URL: http://llvm.org/viewvc/llvm-project?rev=170150view=rev
Log:
[PCH] Make the new PCH format (control block) backwards compatible and
don't crash when loading a PCH with the older format.
The introduction of the control
Author: dblaikie
Date: Thu Dec 13 16:29:06 2012
New Revision: 170154
URL: http://llvm.org/viewvc/llvm-project?rev=170154view=rev
Log:
Debug Info: Emit vtables pointer members as artificial.
I wasn't sure where to put the test case for this, but this seemed like as good
a place as any. I had to
Does dragonegg need to be updated as well?
--paulr
From: cfe-commits-boun...@cs.uiuc.edu [cfe-commits-boun...@cs.uiuc.edu] on
behalf of David Blaikie [dblai...@gmail.com]
Sent: Thursday, December 13, 2012 2:29 PM
To: cfe-commits@cs.uiuc.edu
Subject:
On Thu, Dec 13, 2012 at 2:54 PM, Robinson, Paul
paul.robin...@am.sony.com wrote:
Does dragonegg need to be updated as well?
I don't know, I've not been testing it.
- David
--paulr
From: cfe-commits-boun...@cs.uiuc.edu
Possibly, but dragonegg uses the old debug builder anyhow.
-eric
On Thu, Dec 13, 2012 at 3:00 PM, David Blaikie dblai...@gmail.com wrote:
On Thu, Dec 13, 2012 at 2:54 PM, Robinson, Paul
paul.robin...@am.sony.com wrote:
Does dragonegg need to be updated as well?
I don't know, I've not
Author: efriedma
Date: Thu Dec 13 17:37:17 2012
New Revision: 170160
URL: http://llvm.org/viewvc/llvm-project?rev=170160view=rev
Log:
Make sure the __invoke function for lambdas returns properly. Per bug report
on IRC
Modified:
cfe/trunk/lib/CodeGen/CGClass.cpp
Hi Argyrios,
One of these commits (169830, 169831) added failing test to
clang-X86_64-freebsd builder:
Clang :: Misc/dev-fd-fs.c failed
The builder is here:
http://lab.llvm.org:8011/builders/clang-X86_64-freebsd
For r 169829 all tests pass:
Author: nadav
Date: Thu Dec 13 18:26:15 2012
New Revision: 170164
URL: http://llvm.org/viewvc/llvm-project?rev=170164view=rev
Log:
Make the test less sensitive to the inline threshold.
Modified:
cfe/trunk/test/CodeGenCXX/temp-order.cpp
Modified: cfe/trunk/test/CodeGenCXX/temp-order.cpp
URL:
Eli, it fails in -Asserts build. Not sure why.
define internal void @_ZZ1hvEN3$_78__invokeEv(%struct.A* noalias
sret %agg.result) inlinehint align 2 {
call void @_ZZ1hvENK3$_7clEv(%struct.A* sret %agg.result,
%class.anon.8* undef)
call void @llvm.trap()
unreachable
On Thu, Dec 13, 2012 at 4:29 PM, NAKAMURA Takumi geek4ci...@gmail.com wrote:
Eli, it fails in -Asserts build. Not sure why.
define internal void @_ZZ1hvEN3$_78__invokeEv(%struct.A* noalias
sret %agg.result) inlinehint align 2 {
call void @_ZZ1hvENK3$_7clEv(%struct.A* sret %agg.result,
On Dec 13, 2012, at 4:02 PM, Galina Kistanova gkistan...@gmail.com wrote:
Hi Argyrios,
One of these commits (169830, 169831) added failing test to
clang-X86_64-freebsd builder:
Clang :: Misc/dev-fd-fs.c failed
The builder is here:
http://lab.llvm.org:8011/builders/clang-X86_64-freebsd
All,
The attached patch emits a warning in the frontend if the -isysroot flag is
given an invalid directory. I've implemented this using
llvm::sys:fs::exists(), but please let me know if FileManager::getStatValue()
(or another option) is preferred; I'll modify the patch accordingly. I'll also
2012/12/14 Eli Friedman eli.fried...@gmail.com:
On Thu, Dec 13, 2012 at 4:29 PM, NAKAMURA Takumi geek4ci...@gmail.com wrote:
Eli, it fails in -Asserts build. Not sure why.
define internal void @_ZZ1hvEN3$_78__invokeEv(%struct.A* noalias
sret %agg.result) inlinehint align 2 {
call void
Author: kremenek
Date: Thu Dec 13 19:23:13 2012
New Revision: 170170
URL: http://llvm.org/viewvc/llvm-project?rev=170170view=rev
Log:
Refactor dump methods to make RegionBindingsRef printable in the debugger.
Modified:
cfe/trunk/lib/StaticAnalyzer/Core/RegionStore.cpp
Modified:
Possibly, but dragonegg uses the old debug builder anyhow.
-eric
Ah, check. (Reflexes about changes to interfaces are sometimes hard to
suppress...
I'll try not to twitch in public next time.)
--paulr
___
cfe-commits mailing list
Author: chapuni
Date: Thu Dec 13 20:35:04 2012
New Revision: 170175
URL: http://llvm.org/viewvc/llvm-project?rev=170175view=rev
Log:
clang/test/CodeGenCXX/lambda-expressions.cpp: Relax expression for -Asserts.
entry: is not met in -Asserts build.
Modified:
Author: nico
Date: Thu Dec 13 20:40:09 2012
New Revision: 170176
URL: http://llvm.org/viewvc/llvm-project?rev=170176view=rev
Log:
fix spello
Modified:
cfe/trunk/lib/Parse/ParseTemplate.cpp
Modified: cfe/trunk/lib/Parse/ParseTemplate.cpp
URL:
Author: nico
Date: Thu Dec 13 20:41:18 2012
New Revision: 170177
URL: http://llvm.org/viewvc/llvm-project?rev=170177view=rev
Log:
fix another spello
Modified:
cfe/trunk/include/clang/Parse/Parser.h
Modified: cfe/trunk/include/clang/Parse/Parser.h
URL:
Eli,
I am sorry. My local build tree might be wrong then. I don't reproduce
any odd emissions now.
I reconfirmed r170160 emits as below with -Asserts.
define internal void @_ZZ1hvEN3$_78__invokeEv(%struct.A* noalias sret %agg.res
ult) inlinehint align 2 {
call void
On Thu, Dec 13, 2012 at 5:37 PM, Robinson, Paul
paul.robin...@am.sony.com wrote:
Possibly, but dragonegg uses the old debug builder anyhow.
-eric
Ah, check. (Reflexes about changes to interfaces are sometimes hard to
suppress...
I'll try not to twitch in public next time.)
In this case
On Thu, Dec 13, 2012 at 7:17 PM, David Blaikie dblai...@gmail.com wrote:
On Thu, Dec 13, 2012 at 5:37 PM, Robinson, Paul
paul.robin...@am.sony.com wrote:
Possibly, but dragonegg uses the old debug builder anyhow.
-eric
Ah, check. (Reflexes about changes to interfaces are sometimes
Hi,
C++11 allowed writing vectorvectorint without a space between and
.
Objective-C++11 doesn't seem to allow vectoridprotocol though, and
the diagnostic message clang emits for it even gets worse with -std=c++11
(PR14603).
The attached patch allows vectoridprotocol (only in c++11 mode) by
PR 14529 was opened because neither Clang or LLVM was expanding
calls to creal* or cimag* into instructions that just load the
respective complex field. After some discussion, it was not
considered realistic to do this in LLVM because of the platform
specific way complex types are expanded. Thus
On Thu, Dec 13, 2012 at 7:26 PM, Nico Weber tha...@chromium.org wrote:
Hi,
C++11 allowed writing vectorvectorint without a space between and
.
Objective-C++11 doesn't seem to allow vectoridprotocol though, and the
diagnostic message clang emits for it even gets worse with -std=c++11
On Thu, Dec 13, 2012 at 4:54 PM, Chad Rosier mcros...@apple.com wrote:
All,
The attached patch emits a warning in the frontend if the -isysroot flag
is given an invalid directory. I've implemented this using
llvm::sys:fs::exists(), but please let me know if
FileManager::getStatValue() (or
Author: akirtzidis
Date: Fri Dec 14 00:54:03 2012
New Revision: 170180
URL: http://llvm.org/viewvc/llvm-project?rev=170180view=rev
Log:
Have Sema::ActOnStartOfFunctionDef return the declaration that was passed it.
This fixes the missing warning here:
struct S {
template typename T
void
Author: akirtzidis
Date: Fri Dec 14 00:53:58 2012
New Revision: 170179
URL: http://llvm.org/viewvc/llvm-project?rev=170179view=rev
Log:
Remove code from Sema::ActOnStartOfFunctionTemplateDef that duplicates what
Sema::ActOnStartOfFunctionDef is already doing.
Modified:
55 matches
Mail list logo
