[cfe-commits] r168698 - /cfe/trunk/test/CodeGenCXX/mangle-ms-back-references-pr13207.cpp

2012-11-27 Thread Timur Iskhodzhanov
Author: timu Date: Tue Nov 27 06:55:47 2012 New Revision: 168698 URL: http://llvm.org/viewvc/llvm-project?rev=168698view=rev Log: Remove outdated FIXME; should have removed that in r160782 Modified: cfe/trunk/test/CodeGenCXX/mangle-ms-back-references-pr13207.cpp Modified:

[cfe-commits] [PATCH] Split AddressSanitizer into two passes (FunctionPass, ModulePass), Clang part

2012-11-27 Thread Kostya Serebryany
Hi chandlerc, http://llvm-reviews.chandlerc.com/D138 Files: lib/CodeGen/BackendUtil.cpp Index: lib/CodeGen/BackendUtil.cpp === --- lib/CodeGen/BackendUtil.cpp +++ lib/CodeGen/BackendUtil.cpp @@ -159,7 +159,8 @@ static void

[cfe-commits] r168701 - in /cfe/trunk: docs/UsersManual.html include/clang/Basic/Sanitizers.def lib/CodeGen/CGExprScalar.cpp lib/Driver/SanitizerArgs.h test/CodeGen/catch-undef-behavior.c test/CodeGen

2012-11-27 Thread Will Dietz
Author: wdietz2 Date: Tue Nov 27 09:01:55 2012 New Revision: 168701 URL: http://llvm.org/viewvc/llvm-project?rev=168701view=rev Log: Add -fsanitize=integer for reporting suspicious integer behaviors. Introduces new sanitizer unsigned-integer-overflow. Added:

[cfe-commits] r168703 - in /cfe/trunk: include/clang/Driver/Driver.h lib/Driver/Driver.cpp lib/Frontend/CreateInvocationFromCommandLine.cpp lib/Tooling/Tooling.cpp tools/driver/Makefile tools/driver/d

2012-11-27 Thread Rafael Espindola
Author: rafael Date: Tue Nov 27 10:10:37 2012 New Revision: 168703 URL: http://llvm.org/viewvc/llvm-project?rev=168703view=rev Log: Remove some dead code. CLANG_IS_PRODUCTION is now just a build flag and is not used in any #ifdef. Modified: cfe/trunk/include/clang/Driver/Driver.h

Re: [cfe-commits] [PATCH v2] Set some OpenCl specification mandated types/alignments/etc

2012-11-27 Thread David Tweed
Revised patch actually attached. -Original Message- From: David Tweed [mailto:david.tw...@arm.com] Sent: 27 November 2012 16:09 To: 'Eli Friedman' Cc: Pekka Jääskeläinen; cfe-commits@cs.uiuc.edu Subject: RE: [PATCH v2] Set some OpenCl specification mandated types/alignments/etc Hi Eli,

Re: [cfe-commits] [PATCH v2] Set some OpenCl specification mandated types/alignments/etc

2012-11-27 Thread David Tweed
Hi Eli, thanks for the feedback. In general I've tested everything I know how to test using only the output of a compilation. | Why are you messing with LargeArrayMinWidth and LargeArrayAlign? | Please explain in the patch and add tests. That looks like something I forgot was an implementation

[cfe-commits] r168705 - /cfe/trunk/examples/clang-interpreter/main.cpp

2012-11-27 Thread Rafael Espindola
Author: rafael Date: Tue Nov 27 10:16:02 2012 New Revision: 168705 URL: http://llvm.org/viewvc/llvm-project?rev=168705view=rev Log: Fix examples. Modified: cfe/trunk/examples/clang-interpreter/main.cpp Modified: cfe/trunk/examples/clang-interpreter/main.cpp URL:

[cfe-commits] r168707 - /cfe/trunk/lib/Driver/ToolChains.h

2012-11-27 Thread Chad Rosier
Author: mcrosier Date: Tue Nov 27 11:31:26 2012 New Revision: 168707 URL: http://llvm.org/viewvc/llvm-project?rev=168707view=rev Log: Remove an extra semicolon. Modified: cfe/trunk/lib/Driver/ToolChains.h Modified: cfe/trunk/lib/Driver/ToolChains.h URL:

[cfe-commits] [PATCH] trivial patch for CMake-related build instructions

2012-11-27 Thread Sean McBride
Hi all, Trivial patch to: - add Xcode to the list of project formats CMake can generate - fix spelling: cmake - CMake Cheers, Sean clang-CMake-comments.patch Description: Binary data llvm-CMake-comments.patch Description: Binary data ___

[cfe-commits] [libcxx] r168713 - in /libcxx/trunk/www: index.html libcxx_by_chapter.pdf

2012-11-27 Thread Howard Hinnant
Author: hhinnant Date: Tue Nov 27 12:35:09 2012 New Revision: 168713 URL: http://llvm.org/viewvc/llvm-project?rev=168713view=rev Log: Remove by-chapter breakdown of what is implemented. The chart is now obsolete. Removed: libcxx/trunk/www/libcxx_by_chapter.pdf Modified:

Re: [cfe-commits] [libcxx] r168713 - in /libcxx/trunk/www: index.html libcxx_by_chapter.pdf

2012-11-27 Thread Richard Smith
On 27 Nov 2012 10:37, Howard Hinnant hhinn...@apple.com wrote: Author: hhinnant Date: Tue Nov 27 12:35:09 2012 New Revision: 168713 URL: http://llvm.org/viewvc/llvm-project?rev=168713view=rev Log: Remove by-chapter breakdown of what is implemented. The chart is now obsolete. Removed:

[cfe-commits] [libcxx] r168715 - /libcxx/trunk/www/index.html

2012-11-27 Thread Howard Hinnant
Author: hhinnant Date: Tue Nov 27 12:52:32 2012 New Revision: 168715 URL: http://llvm.org/viewvc/llvm-project?rev=168715view=rev Log: Fix type-o. Modified: libcxx/trunk/www/index.html Modified: libcxx/trunk/www/index.html URL:

Re: [cfe-commits] r168269 - in /cfe/trunk: lib/Lex/Lexer.cpp test/Lexer/eof-char.c test/Lexer/eof-file.c test/Lexer/eof-string.c

2012-11-27 Thread Douglas Gregor
On Nov 17, 2012, at 12:42 PM, Nico Weber tha...@chromium.org wrote: Doug, Pawel marked PR14369 as blocker for the 3.2 release. Is it ok to merge this? (And if so, how do I do that?) Approved. You can merge the patch into the release_32 branch in Subversion. - Doug On Sat, Nov 17,

Re: [cfe-commits] r162937 - in /cfe/trunk: include/clang/Parse/Parser.h lib/Lex/PPMacroExpansion.cpp lib/Parse/ParseExpr.cpp test/SemaCXX/libstdcxx_is_pod_hack.cpp

2012-11-27 Thread Douglas Gregor
On Nov 23, 2012, at 2:02 PM, Andy Gibbs andyg1...@hotmail.co.uk wrote: On Thursday, August 30, 2012 9:04 PM, Douglas Gregor committed: Author: dgregor Date: Thu Aug 30 15:04:43 2012 New Revision: 162937 URL: http://llvm.org/viewvc/llvm-project?rev=162937view=rev Log: Extend the __is_pod

Re: [cfe-commits] r168277 - in /cfe/trunk: include/clang/Basic/DiagnosticCommentKinds.td lib/AST/CommentParser.cpp test/Sema/warn-documentation.cpp

2012-11-27 Thread Douglas Gregor
On Nov 17, 2012, at 4:34 PM, Dmitri Gribenko griboz...@gmail.com wrote: On Sun, Nov 18, 2012 at 2:30 AM, Dmitri Gribenko griboz...@gmail.com wrote: Author: gribozavr Date: Sat Nov 17 18:30:31 2012 New Revision: 168277 URL: http://llvm.org/viewvc/llvm-project?rev=168277view=rev Log:

Re: [cfe-commits] [PATCH] C++11 nullptr-convert standalone tool

2012-11-27 Thread Dmitri Gribenko
I understand that this approach makes edits that are certainly OK from the programmer's point of view. But there is another approach: just walk the AST and find all the null pointers. Did you consider this one? Comment at: nullptr-convert/NullptrActions.h:13 @@ +12,3 @@

Re: [cfe-commits] [PATCH] C++11 nullptr-convert standalone tool

2012-11-27 Thread Edwin Vane
We'll look into searching for all nulls as Richard and Dmitri suggested. http://llvm-reviews.chandlerc.com/D136 ___ cfe-commits mailing list cfe-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Re: [cfe-commits] [PATCH] Fix for PR12222

2012-11-27 Thread Douglas Gregor
On Nov 24, 2012, at 7:29 AM, Erik Verbruggen erik.verbrug...@me.com wrote: Ping? Attached is an update patch against rev. #168535. 0001-PATCH-Fix-for-PR1.patch Commit message says getSourceLocation() when it means getSourceRange(): Changed getLocStart() and getLocEnd() to be

[cfe-commits] r168725 - in /cfe/trunk/lib/Sema: SemaDeclCXX.cpp SemaLookup.cpp

2012-11-27 Thread Richard Smith
Author: rsmith Date: Tue Nov 27 15:20:31 2012 New Revision: 168725 URL: http://llvm.org/viewvc/llvm-project?rev=168725view=rev Log: Simplify checking for whether we should implicitly declare special members and add some assertions. No functionality change. Modified:

[cfe-commits] r168726 - in /cfe/trunk: include/clang/AST/ASTConsumer.h lib/Sema/SemaDecl.cpp unittests/Tooling/ToolingTest.cpp

2012-11-27 Thread Richard Smith
Author: rsmith Date: Tue Nov 27 15:31:01 2012 New Revision: 168726 URL: http://llvm.org/viewvc/llvm-project?rev=168726view=rev Log: Allow an ASTConsumer to selectively skip function bodies while parsing. Patch by Olivier Goffart! Modified: cfe/trunk/include/clang/AST/ASTConsumer.h

Re: [cfe-commits] [patch] Move PrettyStackTraceParserEntry to ParseAST.cpp

2012-11-27 Thread Nico Weber
On Sat, Nov 24, 2012 at 4:16 AM, Dmitri Gribenko griboz...@gmail.com wrote: On Sat, Nov 24, 2012 at 5:40 AM, Nico Weber tha...@chromium.org wrote: On Fri, Nov 23, 2012 at 7:24 PM, Nico Weber tha...@chromium.org wrote: Hi, r128056 moved PrettyStackTraceParserEntry construction from Parser.h

[cfe-commits] r168731 - in /cfe/trunk: include/clang/Parse/Parser.h lib/Parse/ParseAST.cpp lib/Parse/Parser.cpp

2012-11-27 Thread Nico Weber
Author: nico Date: Tue Nov 27 15:57:34 2012 New Revision: 168731 URL: http://llvm.org/viewvc/llvm-project?rev=168731view=rev Log: Move PrettyStackTraceParserEntry to ParseAST.cpp r128056 moved PrettyStackTraceParserEntry construction from Parser.h to ParseAST.cpp, so there's no need to keep this

Re: [cfe-commits] r168277 - in /cfe/trunk: include/clang/Basic/DiagnosticCommentKinds.td lib/AST/CommentParser.cpp test/Sema/warn-documentation.cpp

2012-11-27 Thread Pawel Wodnicki
Hi, On Nov 17, 2012, at 4:34 PM, Dmitri Gribenko griboz...@gmail.com wrote: On Sun, Nov 18, 2012 at 2:30 AM, Dmitri Gribenko griboz...@gmail.com wrote: Author: gribozavr Date: Sat Nov 17 18:30:31 2012 New Revision: 168277 URL: http://llvm.org/viewvc/llvm-project?rev=168277view=rev

Re: [cfe-commits] [Patch] Initial FreeBSD ARM support

2012-11-27 Thread Dmitri Gribenko
On Mon, Nov 26, 2012 at 11:15 AM, Andrew Turner and...@fubar.geek.nz wrote: On Sat, 24 Nov 2012 22:23:40 +0200 Dmitri Gribenko griboz...@gmail.com wrote: On Sat, Nov 24, 2012 at 10:47 AM, Andrew Turner and...@fubar.geek.nz wrote: Hello, The attached patch adds initial support to clang

[cfe-commits] [PATCH] Add three new sanitizers depending on ASan.

2012-11-27 Thread Alexey Samsonov
Hi kcc, rsmith, This patch expose to users three more sanitizers: 1) init-order sanitizer: initialization-order checker. Status: usable, but may produce false positives w/o proper blacklisting. 2) use-after-return sanitizer Status: implemented, but heavily understed. Should be optional, as

Re: [cfe-commits] [PATCH] Support for universal character names in identifiers

2012-11-27 Thread Richard Smith
I had a look at supporting UTF-8 in source files, and came up with the attached approach. getCharAndSize maps UTF-8 characters down to a char with the high bit set, representing the class of the character rather than the character itself. (I've not done any performance measurements yet, and the

Re: [cfe-commits] [PATCH] Support for universal character names in identifiers

2012-11-27 Thread Eli Friedman
On Tue, Nov 27, 2012 at 2:25 PM, Richard Smith rich...@metafoo.co.uk wrote: I had a look at supporting UTF-8 in source files, and came up with the attached approach. getCharAndSize maps UTF-8 characters down to a char with the high bit set, representing the class of the character rather than

[cfe-commits] r168738 - in /cfe/trunk/tools/libclang: CIndex.cpp CIndexer.h

2012-11-27 Thread Argyrios Kyrtzidis
Author: akirtzidis Date: Tue Nov 27 16:49:42 2012 New Revision: 168738 URL: http://llvm.org/viewvc/llvm-project?rev=168738view=rev Log: [libclang] Remove WorkingDir field from CIndexer, it has been rendered useless. No functionality change. Modified: cfe/trunk/tools/libclang/CIndex.cpp

Re: [cfe-commits] [PATCH] Add three new sanitizers depending on ASan.

2012-11-27 Thread Richard Smith
The general approach of exposing more ASan features with -fsanitize= flags seems fine to me, as does making them require -fsanitize=address. The new sanitizer names seem reasonable. I'm not sure whether address-full should include init-order, since the initialization order sanitizer isn't

Re: [cfe-commits] [PATCH] Support for universal character names in identifiers

2012-11-27 Thread Richard Smith
On Tue, Nov 27, 2012 at 2:37 PM, Eli Friedman eli.fried...@gmail.comwrote: On Tue, Nov 27, 2012 at 2:25 PM, Richard Smith rich...@metafoo.co.uk wrote: I had a look at supporting UTF-8 in source files, and came up with the attached approach. getCharAndSize maps UTF-8 characters down to a

[cfe-commits] r168740 - in /cfe/trunk: lib/CodeGen/CGCall.cpp lib/CodeGen/CGExpr.cpp lib/Sema/SemaExpr.cpp lib/Sema/SemaInit.cpp test/CodeGenObjC/arc-blocks.m test/CodeGenObjC/arc-foreach.m test/CodeG

2012-11-27 Thread Fariborz Jahanian
Author: fjahanian Date: Tue Nov 27 17:02:53 2012 New Revision: 168740 URL: http://llvm.org/viewvc/llvm-project?rev=168740view=rev Log: objective-C arc: load of a __weak object happens via call to objc_loadWeak. This retains and autorelease the weakly-refereced object. This hidden autorelease

[cfe-commits] r168741 - in /cfe/trunk: lib/StaticAnalyzer/Core/RegionStore.cpp test/Analysis/misc-ps-region-store.cpp

2012-11-27 Thread Ted Kremenek
Author: kremenek Date: Tue Nov 27 17:05:37 2012 New Revision: 168741 URL: http://llvm.org/viewvc/llvm-project?rev=168741view=rev Log: Provide stop-gap solution to crash reported in PR 14436. This was also covered by rdar://problem/12753384. The static analyzer evaluates a CXXConstructExpr

Re: [cfe-commits] r168740 - in /cfe/trunk: lib/CodeGen/CGCall.cpp lib/CodeGen/CGExpr.cpp lib/Sema/SemaExpr.cpp lib/Sema/SemaInit.cpp test/CodeGenObjC/arc-blocks.m test/CodeGenObjC/arc-foreach.m test/C

2012-11-27 Thread Fariborz Jahanian
On Nov 27, 2012, at 3:02 PM, Fariborz Jahanian wrote: Author: fjahanian Date: Tue Nov 27 17:02:53 2012 New Revision: 168740 URL: http://llvm.org/viewvc/llvm-project?rev=168740view=rev Log: objective-C arc: load of a __weak object happens via call to objc_loadWeak. This retains and

Re: [cfe-commits] [PATCH] Support for universal character names in identifiers

2012-11-27 Thread Eli Friedman
On Tue, Nov 27, 2012 at 3:01 PM, Richard Smith rich...@metafoo.co.uk wrote: On Tue, Nov 27, 2012 at 2:37 PM, Eli Friedman eli.fried...@gmail.com wrote: On Tue, Nov 27, 2012 at 2:25 PM, Richard Smith rich...@metafoo.co.uk wrote: I had a look at supporting UTF-8 in source files, and came up

[cfe-commits] r168745 - /cfe/trunk/www/get_started.html

2012-11-27 Thread NAKAMURA Takumi
Author: chapuni Date: Tue Nov 27 17:34:28 2012 New Revision: 168745 URL: http://llvm.org/viewvc/llvm-project?rev=168745view=rev Log: docs: [CMake] Add Xcode to the list of project formats CMake can generate. Suggested by Sean McBride, thanks! Modified: cfe/trunk/www/get_started.html

[cfe-commits] r168744 - /cfe/trunk/www/get_started.html

2012-11-27 Thread NAKAMURA Takumi
Author: chapuni Date: Tue Nov 27 17:34:08 2012 New Revision: 168744 URL: http://llvm.org/viewvc/llvm-project?rev=168744view=rev Log: clang/www/get_started.html: s/cmake/CMake/ Suggested by Sean McBride, thanks! Modified: cfe/trunk/www/get_started.html Modified:

Re: [cfe-commits] [PATCH] trivial patch for CMake-related build instructions

2012-11-27 Thread NAKAMURA Takumi
Sean, applied in r168744 and r168745, thanks! ...Takumi 2012/11/28 Sean McBride s...@rogue-research.com: Hi all, Trivial patch to: - add Xcode to the list of project formats CMake can generate - fix spelling: cmake - CMake Cheers, Sean ___

Re: [cfe-commits] r168674 - in /cfe/trunk: include/clang/AST/ASTContext.h include/clang/Basic/Builtins.def include/clang/Basic/TargetInfo.h lib/AST/ASTContext.cpp lib/Basic/TargetInfo.cpp lib/Basic/Ta

2012-11-27 Thread Eli Friedman
On Mon, Nov 26, 2012 at 6:58 PM, Eli Friedman eli.fried...@gmail.com wrote: Author: efriedma Date: Mon Nov 26 20:58:24 2012 New Revision: 168674 URL: http://llvm.org/viewvc/llvm-project?rev=168674view=rev Log: Fix the definition of the vfork() builtin on Haiku. PR14378. Modified:

[cfe-commits] [libcxx] r168749 - /libcxx/trunk/test/lit.cfg

2012-11-27 Thread Daniel Dunbar
Author: ddunbar Date: Tue Nov 27 17:56:28 2012 New Revision: 168749 URL: http://llvm.org/viewvc/llvm-project?rev=168749view=rev Log: tests/lit: Change test default parameters to assume local build. - Also, support overriding them with lit parameters. Modified: libcxx/trunk/test/lit.cfg

Re: [cfe-commits] Updated and expanded Clang manual page (docs/tools/clang.pod)

2012-11-27 Thread John McCall
On Nov 21, 2012, at 10:40 AM, Jonathan de Boyne Pollard j.deboynepollard-newsgro...@ntlworld.com wrote: On 2012-11-21 16:19, Chris Lattner wrote: Definitely! Please send patches to the cfe-commits mailing list! The output of svn diff turns out to be larger, at 28KiB, than the file itself,

Re: [cfe-commits] [PATCH] Support for universal character names in identifiers

2012-11-27 Thread Eli Friedman
On Tue, Nov 27, 2012 at 3:33 PM, Eli Friedman eli.fried...@gmail.com wrote: On Tue, Nov 27, 2012 at 3:01 PM, Richard Smith rich...@metafoo.co.uk wrote: On Tue, Nov 27, 2012 at 2:37 PM, Eli Friedman eli.fried...@gmail.com wrote: On Tue, Nov 27, 2012 at 2:25 PM, Richard Smith

[cfe-commits] r168756 - /cfe/trunk/lib/Sema/SemaExprObjC.cpp

2012-11-27 Thread Fariborz Jahanian
Author: fjahanian Date: Tue Nov 27 19:27:44 2012 New Revision: 168756 URL: http://llvm.org/viewvc/llvm-project?rev=168756view=rev Log: objective-C arc: Underline the selector when issuing arc specific diagnostic on the selector. This is objc-arc part of // rdar://11303469 Modified:

[cfe-commits] r168757 - in /cfe/trunk: lib/StaticAnalyzer/Core/ExprEngineC.cpp test/Analysis/misc-ps-region-store.cpp

2012-11-27 Thread Ted Kremenek
Author: kremenek Date: Tue Nov 27 19:49:01 2012 New Revision: 168757 URL: http://llvm.org/viewvc/llvm-project?rev=168757view=rev Log: Fix another false positive due to a CXX temporary object appearing in a C initializer. The stop-gap here is to just drop such objects when processing the

Re: [cfe-commits] r168757 - in /cfe/trunk: lib/StaticAnalyzer/Core/ExprEngineC.cpp test/Analysis/misc-ps-region-store.cpp

2012-11-27 Thread Jordan Rose
Should we revert the previous fix in RegionStore, then? On Nov 27, 2012, at 17:49 , Ted Kremenek kreme...@apple.com wrote: Author: kremenek Date: Tue Nov 27 19:49:01 2012 New Revision: 168757 URL: http://llvm.org/viewvc/llvm-project?rev=168757view=rev Log: Fix another false positive due

Re: [cfe-commits] [cfe-dev] Allow to selectively skip function bodies while parsing.

2012-11-27 Thread Olivier Goffart
On Monday 26 November 2012 20:54:28 Richard Smith wrote: On Mon, Nov 26, 2012 at 4:03 AM, Olivier Goffart ogoff...@kde.org wrote: Hi, I would like to upstream the attached patch which allows ASTConsumer to select which function to skip while parsing. I have been using it to do a

[cfe-commits] scan-build not detecting .C files (capital C).

2012-11-27 Thread Peeter Joot
Not knowing your patch protocol, I posted a small script patch to your bugilla for the ccc-analyzer script that's part of tools/clang/tools/scan-build http://llvm.org/bugs/show_bug.cgi?id=14443 Some rationale and discussion can be found in the bug entry above. I've attached the patch for review

[cfe-commits] r168771 - in /cfe/trunk: include/clang/AST/ASTUnresolvedSet.h include/clang/AST/ASTVector.h include/clang/AST/DeclCXX.h include/clang/AST/Expr.h include/clang/AST/UnresolvedSet.h include

2012-11-27 Thread Argyrios Kyrtzidis
Author: akirtzidis Date: Tue Nov 27 21:56:16 2012 New Revision: 168771 URL: http://llvm.org/viewvc/llvm-project?rev=168771view=rev Log: Introduce ASTUnresolvedSet, an UnresolvedSet-like class, whose contents are allocated using the allocator associated with an ASTContext. Use this inside

Re: [cfe-commits] r168769 - in /cfe/trunk: include/clang/Basic/DiagnosticGroups.td include/clang/Basic/DiagnosticSemaKinds.td lib/Sema/SemaDeclCXX.cpp test/CXX/dcl.dcl/dcl.spec/dcl.constexpr/p4.cpp te

2012-11-27 Thread John McCall
On Nov 27, 2012, at 7:45 PM, Richard Smith richard-l...@metafoo.co.uk wrote: Author: rsmith Date: Tue Nov 27 21:45:24 2012 New Revision: 168769 URL: http://llvm.org/viewvc/llvm-project?rev=168769view=rev Log: C++ core issue 1344, PR10618: promote addition of default argument makes this a

Re: [cfe-commits] r168769 - in /cfe/trunk: include/clang/Basic/DiagnosticGroups.td include/clang/Basic/DiagnosticSemaKinds.td lib/Sema/SemaDeclCXX.cpp test/CXX/dcl.dcl/dcl.spec/dcl.constexpr/p4.cpp te

2012-11-27 Thread Richard Smith
On Tue, Nov 27, 2012 at 8:10 PM, John McCall rjmcc...@apple.com wrote: On Nov 27, 2012, at 7:45 PM, Richard Smith richard-l...@metafoo.co.uk wrote: Author: rsmith Date: Tue Nov 27 21:45:24 2012 New Revision: 168769 URL: http://llvm.org/viewvc/llvm-project?rev=168769view=rev Log:

Re: [cfe-commits] r168757 - in /cfe/trunk: lib/StaticAnalyzer/Core/ExprEngineC.cpp test/Analysis/misc-ps-region-store.cpp

2012-11-27 Thread Ted Kremenek
Yes. I've gone and done that. On Nov 27, 2012, at 5:54 PM, Jordan Rose jordan_r...@apple.com wrote: Should we revert the previous fix in RegionStore, then? On Nov 27, 2012, at 17:49 , Ted Kremenek kreme...@apple.com wrote: Author: kremenek Date: Tue Nov 27 19:49:01 2012 New Revision:

[cfe-commits] r168774 - /cfe/trunk/lib/StaticAnalyzer/Core/RegionStore.cpp

2012-11-27 Thread Ted Kremenek
Author: kremenek Date: Tue Nov 27 23:36:28 2012 New Revision: 168774 URL: http://llvm.org/viewvc/llvm-project?rev=168774view=rev Log: Remove workaround in RegionStore in r168741 since it is handled more generally by r168757. Modified: cfe/trunk/lib/StaticAnalyzer/Core/RegionStore.cpp

[cfe-commits] [PATCH] Refactor -fsanitize arguments parsing: provide a more careful diagnostic.

2012-11-27 Thread Alexey Samsonov
Hi rsmith, This patch improves diagnostic messages printed for invalid sets of values in -fsanitize= flags. See regression test diff for examples. http://llvm-reviews.chandlerc.com/D143 Files: test/Driver/fsanitize.c lib/Driver/Tools.cpp lib/Driver/SanitizerArgs.h Index:

[cfe-commits] r168775 - in /cfe/trunk: include/clang/AST/DeclCXX.h lib/AST/ASTImporter.cpp lib/AST/DeclCXX.cpp lib/Sema/SemaDeclCXX.cpp lib/Serialization/ASTReaderDecl.cpp lib/Serialization/ASTWriter.

2012-11-27 Thread Richard Smith
Author: rsmith Date: Wed Nov 28 00:23:12 2012 New Revision: 168775 URL: http://llvm.org/viewvc/llvm-project?rev=168775view=rev Log: Store on the CXXRecordDecl whether the class has, or would have, a copy constructor/assignment operator with a const-qualified parameter type. The prior method for

Re: [cfe-commits] r168674 - in /cfe/trunk: include/clang/AST/ASTContext.h include/clang/Basic/Builtins.def include/clang/Basic/TargetInfo.h lib/AST/ASTContext.cpp lib/Basic/TargetInfo.cpp lib/Basic/Ta

2012-11-27 Thread Douglas Gregor
On Nov 27, 2012, at 3:45 PM, Eli Friedman eli.fried...@gmail.com wrote: On Mon, Nov 26, 2012 at 6:58 PM, Eli Friedman eli.fried...@gmail.com wrote: Author: efriedma Date: Mon Nov 26 20:58:24 2012 New Revision: 168674 URL: http://llvm.org/viewvc/llvm-project?rev=168674view=rev Log: Fix

Re: [cfe-commits] [PATCH] Add three new sanitizers depending on ASan.

2012-11-27 Thread Alexey Samsonov
Ok. Kostya, what is your opinion? Comment at: lib/Driver/Tools.cpp:1518-1523 @@ -1517,1 +1517,8 @@ + + // If -fsanitize contains extra features of ASan, it should also + // explicitly contain -fsanitize=address. + if (NeedsAsan ((Kind Address) == 0)) +

Re: [cfe-commits] [PATCH] Refactor -fsanitize arguments parsing: provide a more careful diagnostic.

2012-11-27 Thread Richard Smith
Comment at: lib/Driver/SanitizerArgs.h:137-140 @@ +136,6 @@ + I != E; ++I) { + unsigned Add, Remove; + if (parse(D, Args, *I, Add, Remove, false) + (Add Kind)) +return describeSanitizeArg(Args, *I, Kind); +} The contract

Re: [cfe-commits] [PATCH] Array exception type decay to pointer automatically

2012-11-27 Thread 谷汶翰
Hi Richard, I've revised it again. I think this time all similar cases can run. Please have a look. http://llvm.org/bugs/attachment.cgi?id=9601action=diff PR14388: http://llvm.org/bugs/show_bug.cgi?id=14388 Thanks! 2012/11/22 Richard Smith rich...@metafoo.co.uk Hi, On Wed, Nov 21, 2012 at

Re: [cfe-commits] [PATCH] Array exception type decay to pointer automatically

2012-11-27 Thread 谷汶翰
Wrong file, sorry. http://llvm.org/bugs/attachment.cgi?id=9602action=diff Thanks. 2012/11/28 WenHan Gu (谷汶翰) wenhan...@gmail.com Hi Richard, I've revised it again. I think this time all similar cases can run. Please have a look. http://llvm.org/bugs/attachment.cgi?id=9601action=diff

[cfe-commits] r168776 - in /cfe/trunk: lib/Lex/Lexer.cpp test/CXX/lex/lex.literal/lex.ext/p5.cpp test/CodeGen/string-literal.c

2012-11-27 Thread Richard Smith
Author: rsmith Date: Wed Nov 28 01:29:00 2012 New Revision: 168776 URL: http://llvm.org/viewvc/llvm-project?rev=168776view=rev Log: Teach Lexer::getSpelling about raw string literals. Specifically, if a raw string literal needs cleaning (because it contains line-splicing in the encoding prefix or

Re: [cfe-commits] [PATCH] Array exception type decay to pointer automatically

2012-11-27 Thread Richard Smith
On Tue, Nov 27, 2012 at 11:29 PM, WenHan Gu (谷汶翰) wenhan...@gmail.comwrote: Wrong file, sorry. http://llvm.org/bugs/attachment.cgi?id=9602action=diff Thanks! +/// @param[in,out] T It will be decayed to a pointer automatically when input Please use \param, not @param, to match the