Author: marshall
Date: Sun Oct 25 01:02:23 2015
New Revision: 251220
URL: http://llvm.org/viewvc/llvm-project?rev=251220&view=rev
Log:
Update C++ status from Kona
Modified:
libcxx/trunk/www/cxx1z_status.html
libcxx/trunk/www/kona.html
Modified: libcxx/trunk/www/cxx1z_status.html
URL:
ht
Author: dcoughlin
Date: Sat Oct 24 20:30:18 2015
New Revision: 251218
URL: http://llvm.org/viewvc/llvm-project?rev=251218&view=rev
Log:
[analyzer] scan-build: Teach ccc-analyzer about -Xclang.
Update ccc-analyzer to forward both -Xclang and its following argument to the
the compiler driver. Previ
Author: rafael
Date: Sat Oct 24 18:15:31 2015
New Revision: 251214
URL: http://llvm.org/viewvc/llvm-project?rev=251214&view=rev
Log:
Simplify boolean conditional return statements in lib/Basic.
Patch by Richard.
Modified:
cfe/trunk/lib/Basic/Targets.cpp
Modified: cfe/trunk/lib/Basic/Targets
r251213
On 24 October 2015 at 14:58, Richard wrote:
> LegalizeAdulthood updated this revision to Diff 38317.
> LegalizeAdulthood added a comment.
>
> Update to latest.
> I do not have commit access.
>
>
> http://reviews.llvm.org/D10012
>
> Files:
> lib/CodeGen/BranchFolding.cpp
> lib/CodeGen/
LegalizeAdulthood updated this revision to Diff 38324.
LegalizeAdulthood added a comment.
Update from latest.
Update from comments.
I do not have commit access.
http://reviews.llvm.org/D10019
Files:
lib/Sema/SemaChecking.cpp
lib/Sema/SemaCodeComplete.cpp
lib/Sema/SemaDecl.cpp
lib/Sema/S
Author: piotrdz
Date: Sat Oct 24 15:11:47 2015
New Revision: 251204
URL: http://llvm.org/viewvc/llvm-project?rev=251204&view=rev
Log:
Test commit
Modified:
clang-tools-extra/trunk/clang-tidy/readability/InconsistentDeclarationParameterNameCheck.cpp
Modified:
clang-tools-extra/trunk/clang-t
LegalizeAdulthood added a comment.
In http://reviews.llvm.org/D10014#178294, @calabrese wrote:
> I like the idea of all of these changes (this change list and the others),
> but just a suggestion -- it looks like when the branch of the original
> condition returns false, as was the case in the
LegalizeAdulthood updated this revision to Diff 38323.
LegalizeAdulthood added a comment.
Update from latest.
I do not have commit access.
http://reviews.llvm.org/D10014
Files:
lib/Edit/RewriteObjCFoundationAPI.cpp
Index: lib/Edit/RewriteObjCFoundationAPI.cpp
LegalizeAdulthood updated this revision to Diff 38322.
LegalizeAdulthood added a comment.
Update to latest.
I do not have commit access.
http://reviews.llvm.org/D10017
Files:
lib/Lex/PPDirectives.cpp
lib/Lex/PPMacroExpansion.cpp
Index: lib/Lex/PPMacroExpansion.cpp
=
LegalizeAdulthood updated this revision to Diff 38321.
LegalizeAdulthood added a comment.
Update from latest.
I do not have commit access.
http://reviews.llvm.org/D10010
Files:
lib/AST/ASTContext.cpp
lib/AST/ASTDiagnostic.cpp
lib/AST/Decl.cpp
lib/AST/DeclCXX.cpp
lib/AST/Expr.cpp
lib
LegalizeAdulthood added inline comments.
Comment at: lib/StaticAnalyzer/Core/ExprEngineCallAndReturn.cpp:692-693
@@ -691,4 +691,2 @@
CXXBasePaths Paths(false, false, false);
- if (RD->lookupInBases(
- [DeclName](const CXXBaseSpecifier *Specifier, CXXBasePath &Path)
LegalizeAdulthood updated this revision to Diff 38319.
LegalizeAdulthood added a comment.
Update from latest
http://reviews.llvm.org/D10022
Files:
lib/StaticAnalyzer/Core/ExprEngineCallAndReturn.cpp
lib/StaticAnalyzer/Core/SValBuilder.cpp
lib/StaticAnalyzer/Core/SymbolManager.cpp
Index:
LegalizeAdulthood updated this revision to Diff 38318.
LegalizeAdulthood added a comment.
Update to latest.
I do not have commit access.
http://reviews.llvm.org/D10011
Files:
lib/Basic/Targets.cpp
Index: lib/Basic/Targets.cpp
==
LegalizeAdulthood updated this revision to Diff 38317.
LegalizeAdulthood added a comment.
Update to latest.
I do not have commit access.
http://reviews.llvm.org/D10012
Files:
lib/CodeGen/BranchFolding.cpp
lib/CodeGen/CodeGenPrepare.cpp
lib/CodeGen/LiveDebugVariables.cpp
lib/CodeGen/Mach
LegalizeAdulthood updated this revision to Diff 38315.
LegalizeAdulthood marked 2 inline comments as done.
LegalizeAdulthood added a comment.
Update to latest.
Update from comments.
I don't have commit access.
http://reviews.llvm.org/D10024
Files:
tools/libclang/CIndex.cpp
tools/libclang/In
Hi Eric,
Understood, thanks!
Michael
> On Oct 24, 2015, at 1:18 AM, Eric Fiselier wrote:
>
> Hi Michael,
>
> Sorry I'm holding this patch up in review. The fix is quite "novel" and I
> want to make sure we get it right. If we can't land it over the weekend I'll
> ask Richard to revert while
As requested on cfe-dev, enclosed is a patch (that needs some
style-related work) to calculate the complete fully qualified name of
arbitrary Qualtypes, including recursively expanding template
parameters and subtypes.
Seems to me like it follows most of the conventions described here:
http://llv
Unfortunately the asm label attribute doesn’t seem to work in this case, at
least on Darwin. I get two symbols in the IR, with the asm label having the \01
prefix from MangleContext::mangleName():
@"\01OBJC_CLASS_$_NSNumber" = common global %struct.art_class* null, align 8
slavapestov added a subscriber: slavapestov.
slavapestov added a comment.
Unfortunately the asm label attribute doesn’t seem to work in this case, at
least on Darwin. I get two symbols in the IR, with the asm label having the \01
prefix from MangleContext::mangleName():
@"\01OBJC_CLASS_$_NSNumb
LegalizeAdulthood added a comment.
I do not have commit access; someone will need to commit this for me.
http://reviews.llvm.org/D10025
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
LegalizeAdulthood updated this revision to Diff 38313.
LegalizeAdulthood added a comment.
Update to latest
http://reviews.llvm.org/D10025
Files:
clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp
Index: clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp
==
LegalizeAdulthood updated this revision to Diff 38312.
LegalizeAdulthood added a comment.
Update to latest.
This changeset needs reveiw; I am unsure of who to add as specific reviewers.
http://reviews.llvm.org/D10023
Files:
lib/StaticAnalyzer/Frontend/CheckerRegistration.cpp
Index: lib/Stati
LegalizeAdulthood updated this revision to Diff 38311.
LegalizeAdulthood added a comment.
Update to latest.
This changeset needs review; I am unsure who to add as specific reviewers.
http://reviews.llvm.org/D10021
Files:
lib/StaticAnalyzer/Checkers/BasicObjCFoundationChecks.cpp
lib/StaticAn
LegalizeAdulthood added a comment.
This changeset needs review, but I'm unsure of who to add as explicit reviewers.
http://reviews.llvm.org/D10020
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo
LegalizeAdulthood updated this revision to Diff 38310.
LegalizeAdulthood added a comment.
Update to latest
http://reviews.llvm.org/D10020
Files:
lib/Serialization/ASTReader.cpp
lib/Serialization/ASTWriter.cpp
Index: lib/Serialization/ASTWriter.cpp
==
LegalizeAdulthood added a comment.
This changeset needs review; I'm unsure of who to add as explicit reviewers.
http://reviews.llvm.org/D10016
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe
LegalizeAdulthood updated this revision to Diff 38309.
LegalizeAdulthood added a comment.
Update to latest
http://reviews.llvm.org/D10016
Files:
lib/Frontend/VerifyDiagnosticConsumer.cpp
Index: lib/Frontend/VerifyDiagnosticConsumer.cpp
a.sidorin added inline comments.
Comment at: lib/StaticAnalyzer/Checkers/VforkChecker.cpp:47
@@ +46,3 @@
+
+ bool isChildProcess(const ProgramStateRef State) const;
+
ygribov wrote:
> a.sidorin wrote:
> > I think it's a good idea to make some functions static and
ygribov added inline comments.
Comment at: lib/StaticAnalyzer/Checkers/VforkChecker.cpp:149
@@ +148,3 @@
+
+ // see if it's an ordinary assignment
+ do {
ygribov wrote:
> a.sidorin wrote:
> > You can use early return to escape do{}.
> In this particular case - y
ygribov added inline comments.
Comment at: lib/StaticAnalyzer/Checkers/VforkChecker.cpp:47
@@ +46,3 @@
+
+ bool isChildProcess(const ProgramStateRef State) const;
+
a.sidorin wrote:
> I think it's a good idea to make some functions static and/or move them out
>
a.sidorin added a comment.
I put some suggestions in inline comments.
Comment at: lib/StaticAnalyzer/Checkers/VforkChecker.cpp:47
@@ +46,3 @@
+
+ bool isChildProcess(const ProgramStateRef State) const;
+
I think it's a good idea to make some functions static an
Hi Michael,
Sorry I'm holding this patch up in review. The fix is quite "novel" and I
want to make sure we get it right. If we can't land it over the weekend
I'll ask Richard to revert while we work on it.
/Eric
On Oct 23, 2015 10:13 PM, "Michael Zolotukhin via cfe-commits" <
cfe-commits@lists.ll
Hi Richard,
Is this patch ready for commit, or were you just checking an idea? Our bots are
still failing to build povray, so we’re really looking forward for some fix:)
Thanks,
Michael
> On Oct 15, 2015, at 6:21 PM, Manman Ren via cfe-commits
> wrote:
>
>>
>> On Oct 15, 2015, at 1:41 PM, R
Author: chapuni
Date: Sat Oct 24 02:16:41 2015
New Revision: 251182
URL: http://llvm.org/viewvc/llvm-project?rev=251182&view=rev
Log:
clang/module.modulemap: s/Basic/BuiltinsR600.def/Basic/BuiltinsAMDGPU.def/
Modified:
cfe/trunk/include/clang/module.modulemap
Modified: cfe/trunk/include/clan
34 matches
Mail list logo
