Author: ericwf
Date: Sat Sep 5 01:57:29 2015
New Revision: 246914
URL: http://llvm.org/viewvc/llvm-project?rev=246914&view=rev
Log:
Try and fix broken bugzilla link
Modified:
libcxx/trunk/docs/index.rst
Modified: libcxx/trunk/docs/index.rst
URL:
http://llvm.org/viewvc/llvm-project/libcxx/t
Author: ericwf
Date: Sat Sep 5 01:50:03 2015
New Revision: 246913
URL: http://llvm.org/viewvc/llvm-project?rev=246913&view=rev
Log:
Cleanup new documentation index and transfer more information from
www/index.html
Modified:
libcxx/trunk/docs/index.rst
Modified: libcxx/trunk/docs/index.rst
steven_wu added a subscriber: steven_wu.
steven_wu added a comment.
This commit seems to cause miscompile in LNT testsuite with -O0 and -O3
http://lab.llvm.org:8080/green/job/perf_o0g_run/7070/warnings2Result/new/
http://lab.llvm.org:8080/green/job/perf_o3lto_run/15591/warnings2Result/new/
Looks
Author: tbrethou
Date: Sat Sep 5 00:38:50 2015
New Revision: 246912
URL: http://llvm.org/viewvc/llvm-project?rev=246912&view=rev
Log:
Remove test commit.
Modified:
libcxx/trunk/docs/README.txt
Modified: libcxx/trunk/docs/README.txt
URL:
http://llvm.org/viewvc/llvm-project/libcxx/trunk/docs
Author: tbrethou
Date: Sat Sep 5 00:34:27 2015
New Revision: 246911
URL: http://llvm.org/viewvc/llvm-project?rev=246911&view=rev
Log:
Test temporary commit.
Modified:
libcxx/trunk/docs/README.txt
Modified: libcxx/trunk/docs/README.txt
URL:
http://llvm.org/viewvc/llvm-project/libcxx/trunk/
Author: ericwf
Date: Sat Sep 5 00:29:23 2015
New Revision: 246910
URL: http://llvm.org/viewvc/llvm-project?rev=246910&view=rev
Log:
Test commit for builder
Modified:
libcxx/trunk/docs/Makefile.sphinx
libcxx/trunk/docs/index.rst
Modified: libcxx/trunk/docs/Makefile.sphinx
URL:
http://ll
Author: ericwf
Date: Sat Sep 5 00:12:04 2015
New Revision: 246909
URL: http://llvm.org/viewvc/llvm-project?rev=246909&view=rev
Log:
Add temporary Makefile.sphinx build file to get libcxx.llvm.org/docs going
Added:
libcxx/trunk/docs/Makefile.sphinx
Added: libcxx/trunk/docs/Makefile.sphinx
UR
On Fri, Aug 07, 2015 at 11:35:26PM -0700, Omar Sandoval wrote:
> This makes it easier for tools using the Python libclang bindings to
> display diagnostics in a manner consistent with clang.
Ping, any comments on this?
Thanks.
> ---
> bindings/python/clang/cindex.py | 33 +++
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
This looks fine, just make sure the indentation is right. Phab seems confused.
Repository:
rL LLVM
http://reviews.llvm.org/D12466
___
cfe-commits ma
Author: ericwf
Date: Fri Sep 4 20:32:48 2015
New Revision: 246906
URL: http://llvm.org/viewvc/llvm-project?rev=246906&view=rev
Log:
Try building docs again.
Modified:
libcxx/trunk/docs/index.rst
Modified: libcxx/trunk/docs/index.rst
URL:
http://llvm.org/viewvc/llvm-project/libcxx/trunk/doc
hubert.reinterpretcast accepted this revision.
hubert.reinterpretcast added a comment.
This revision is now accepted and ready to land.
LGTM
http://reviews.llvm.org/D12435
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org
eugenis added inline comments.
Comment at: lib/CodeGen/CodeGenModule.h:505
@@ +504,3 @@
+
+ llvm::SmallVector AlwaysInlineFunctions;
+
Right. Sorry I missed the comment.
I've switched this to a vector.
Repository:
rL LLVM
http://reviews.llvm.org/D12087
__
eugenis updated this revision to Diff 34095.
Repository:
rL LLVM
http://reviews.llvm.org/D12087
Files:
lib/CodeGen/CGCXX.cpp
lib/CodeGen/CGClass.cpp
lib/CodeGen/CGOpenMPRuntime.cpp
lib/CodeGen/CodeGenModule.cpp
lib/CodeGen/CodeGenModule.h
lib/CodeGen/ItaniumCXXABI.cpp
test/CodeGe
rsmith added inline comments.
Comment at: lib/CodeGen/CodeGenModule.h:505
@@ +504,3 @@
+
+ llvm::SetVector AlwaysInlineFunctions;
+
Did you try making this a vector? It'd be nice to avoid the set overhead here
if we can.
Repository:
rL LLVM
http://reviews.l
eugenis accepted this revision.
eugenis added a comment.
This revision is now accepted and ready to land.
I'd call the test sanitize-dtor-trivial.cpp
http://reviews.llvm.org/D12616
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists
nmusgrave updated this revision to Diff 34094.
nmusgrave marked an inline comment as done.
nmusgrave added a comment.
- Only check simplest object for existence of sanitizing callback.
http://reviews.llvm.org/D12616
Files:
test/CodeGenCXX/sanitize-dtor-generated.cpp
Index: test/CodeGenCXX/sa
rsmith accepted this revision.
Comment at: lib/AST/ItaniumMangle.cpp:504
@@ +503,3 @@
+ if (FD->hasAttr() && FD->hasAttr())
+Out << "Ua6deviceUa4host";
+
If we have `enable_if` on a `__host__` `__device__` function, I think we should
mangle as `Ua6deviceUa9e
Author: ericwf
Date: Fri Sep 4 18:19:11 2015
New Revision: 246895
URL: http://llvm.org/viewvc/llvm-project?rev=246895&view=rev
Log:
Test commit for sphinx docs try2
Modified:
libcxx/trunk/docs/index.rst
Modified: libcxx/trunk/docs/index.rst
URL:
http://llvm.org/viewvc/llvm-project/libcxx/t
EricWF added a comment.
@mclow.lists ping. Hopefully this should be a pretty simple review.
http://reviews.llvm.org/D11397
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: ericwf
Date: Fri Sep 4 17:57:00 2015
New Revision: 246893
URL: http://llvm.org/viewvc/llvm-project?rev=246893&view=rev
Log:
Test commit to see if libcxx.llvm.org/docs builds
Modified:
libcxx/trunk/TODO.TXT
Modified: libcxx/trunk/TODO.TXT
URL:
http://llvm.org/viewvc/llvm-project/lib
On Fri, Aug 14, 2015 at 2:55 AM, Manuel Klimek via cfe-commits
wrote:
> Author: klimek
> Date: Fri Aug 14 04:55:36 2015
> New Revision: 245036
>
> URL: http://llvm.org/viewvc/llvm-project?rev=245036&view=rev
> Log:
> Add structed way to express command line options in the compilation database.
>
>
xazax.hun created this revision.
xazax.hun added reviewers: dcoughlin, zaks.anna, jordan_rose, ted.
xazax.hun added a subscriber: cfe-commits.
This patch adds lambda support for the static analyzer.
All of my initial tests are passed.
Before this patch each time a LambdaExpr was encountered a sin
Author: gbiv
Date: Fri Sep 4 17:36:18 2015
New Revision: 246890
URL: http://llvm.org/viewvc/llvm-project?rev=246890&view=rev
Log:
Fix a bug in __builtin_object_size cast removal
Apparently there are many cast kinds that may cause implicit pointer
arithmetic to happen. In light of this, the cast
Author: rnk
Date: Fri Sep 4 17:32:51 2015
New Revision: 246889
URL: http://llvm.org/viewvc/llvm-project?rev=246889&view=rev
Log:
Relax partial-init test case for ARM
Modified:
cfe/trunk/test/CodeGenCXX/partial-init.cpp
Modified: cfe/trunk/test/CodeGenCXX/partial-init.cpp
URL:
http://llvm.o
Oh, nice so this was actually done by Kuba to fix sanitizer unittests:
"echo '#include ' | ${COMPILER_RT_TEST_COMPILER} -E -x c++
- > /dev/null"
"if [ $? != 0 ] "
" then echo"
" echo 'Your just-built clang cannot find C++ headers, which are
needed to build and run compile
ismailp added a comment.
Thank you for the feedback! I have added the test in a C file, so we can get
the first part of the patch in -- between lines 778-780.
I think self-loop is a different problem, and that requires its own patch. I'll
see what I can do about self-loop, if someone else isn't
ismailp updated this revision to Diff 34082.
ismailp added a comment.
- Rolled back to the first version of patch, where only parameter `Out` is
renamed to `OutStream`
- Use a different test that doesn't trigger the self-loop assertion.
http://reviews.llvm.org/D12119
Files:
lib/StaticAnalyze
Author: hfinkel
Date: Fri Sep 4 16:49:21 2015
New Revision: 246882
URL: http://llvm.org/viewvc/llvm-project?rev=246882&view=rev
Log:
Don't crash on a self-alias declaration
We were crashing in CodeGen given input like this:
int self_alias(void) __attribute__((weak, alias("self_alias")));
suc
Author: rsmith
Date: Fri Sep 4 16:44:32 2015
New Revision: 246881
URL: http://llvm.org/viewvc/llvm-project?rev=246881&view=rev
Log:
Fix crash on invalid if we can't find a suitable PCH file in a specified
directory, and our frontend action cares whether the frontend setup actually
succeeded.
Mod
jroelofs added a comment.
I trust your judgement.
http://reviews.llvm.org/D12508
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
alexfh added a comment.
Sorry for the long delay.
This version looks significantly better. Thank you for the updates!
Comment at:
clang-tidy/readability/InconsistentDeclarationParameterNameCheck.cpp:1
@@ +1,2 @@
+//===--- InconsistentDeclarationParameterNameCheck.cpp -
+//clan
This revision was automatically updated to reflect the committed changes.
Closed by commit rL246880: Don't use unreachable as a placeholder, it confuses
EmitBlock (authored by rnk).
Changed prior to commit:
http://reviews.llvm.org/D12412?vs=33347&id=34080#toc
Repository:
rL LLVM
http://revi
EricWF added inline comments.
Comment at: utils/gen_link_script/gen_link_script.py:1
@@ +1,2 @@
+#!/usr/bin/env python
+import os
jroelofs wrote:
> Do you foresee these linker scripts being significantly more complicated for
> other platforms? ISTM that it would
Author: rnk
Date: Fri Sep 4 16:39:15 2015
New Revision: 246880
URL: http://llvm.org/viewvc/llvm-project?rev=246880&view=rev
Log:
Don't use unreachable as a placeholder, it confuses EmitBlock
This fixes an issue raised in D12412, where we generated invalid IR.
Thanks to Vedant Kumar for coming u
Author: angelgarcia
Date: Fri Sep 4 16:37:05 2015
New Revision: 246879
URL: http://llvm.org/viewvc/llvm-project?rev=246879&view=rev
Log:
Avoid repeated replacements on loop-convert check.
Summary: The InitListExpr subtree is visited twice, this caused the check to do
multiple replacements. Adde
george.burgess.iv closed this revision.
george.burgess.iv marked 2 inline comments as done.
george.burgess.iv added a comment.
r246877. Thanks for the review!
(Also: Forgot to submit comments with the most recent revision, so you're
getting them all now. Sorry. :) )
Comment at
Author: gbiv
Date: Fri Sep 4 16:28:13 2015
New Revision: 246877
URL: http://llvm.org/viewvc/llvm-project?rev=246877&view=rev
Log:
Increase accuracy of __builtin_object_size.
Improvements:
- For all types, we would give up in a case such as:
__builtin_object_size((char*)&foo, N);
even if w
EricWF added a comment.
In http://reviews.llvm.org/D12512#237175, @joerg wrote:
> Please don't commit this as is. Many platforms have posix_memalign or
> equivalent, which makes this both simpler and potentially without wasting
> memory. Compare e.g. http://reviews.llvm.org/D12001.
Will do. A
EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.
LGTM modulo handling C++03. Currently compiles w/
clang in C++03. Please either ensure your portions of the searchers also
compile in C++03 and change the tests or #ifdef them out before comm
rnk added a subscriber: rnk.
rnk added a comment.
I think a better fix would be to insert a non-terminator placeholder that
doesn't confuse EmitBlock.
http://reviews.llvm.org/D12412
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lis
angelgarcia updated this revision to Diff 34076.
angelgarcia added a comment.
Wooops, solved.
http://reviews.llvm.org/D12631
Files:
clang-tidy/modernize/LoopConvertUtils.cpp
clang-tidy/modernize/LoopConvertUtils.h
test/clang-tidy/modernize-loop-convert-extra.cpp
Index: test/clang-tidy/mo
alexfh accepted this revision.
alexfh added a reviewer: alexfh.
alexfh added a comment.
This revision is now accepted and ready to land.
Looks good with a comment.
Comment at: clang-tidy/modernize/LoopConvertUtils.h:211
@@ -210,3 +210,3 @@
/// \brief A class to encapsulate lowe
On 9/4/15 2:55 PM, Yuri Gribov wrote:
On Fri, Sep 4, 2015 at 11:49 PM, Jonathan Roelofs
mailto:jonat...@codesourcery.com>> wrote:
On 9/4/15 2:45 PM, Yuri Gribov via cfe-commits wrote:
Hi all,
According to https://llvm.org/bugs/show_bug.cgi?id=17821 , when
Clang
On Fri, Sep 4, 2015 at 11:49 PM, Jonathan Roelofs wrote:
> On 9/4/15 2:45 PM, Yuri Gribov via cfe-commits wrote:
>
>> Hi all,
>>
>> According to https://llvm.org/bugs/show_bug.cgi?id=17821 , when Clang
>> gets built on OS X we need to build it with libcxx and libcxxabi
>> (otherwise tests will fa
steven_wu updated this revision to Diff 34073.
steven_wu added a comment.
Sorry, made a typo on the previous patch.
A little background information. When the greendragon jenkins make an
system upgrade in the future, it will need this flag as well to build and
test libcxx through cmake/ninja.
ht
hintonda created this revision.
hintonda added a reviewer: rsmith.
hintonda added a subscriber: cfe-commits.
Current behavior doesn't add c++ header path if libc++ wasn't installed via
macports in /usr. This change will try to locate c++ headers from various
locations, including in tree, via ma
EricWF added a reviewer: danalbert.
EricWF added a subscriber: danalbert.
EricWF added a comment.
@danalbert actually uses this option. I would like him to confirm this change.
http://reviews.llvm.org/D12644
___
cfe-commits mailing list
cfe-commits@
On 9/4/15 2:45 PM, Yuri Gribov via cfe-commits wrote:
Hi all,
According to https://llvm.org/bugs/show_bug.cgi?id=17821 , when Clang
gets built on OS X we need to build it with libcxx and libcxxabi
(otherwise tests will fail to link with "can't find iostream". This
patch adds a note to getting
vsk added a comment.
Ping
http://reviews.llvm.org/D12412
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Hi all,
According to https://llvm.org/bugs/show_bug.cgi?id=17821 , when Clang gets
built on OS X we need to build it with libcxx and libcxxabi (otherwise
tests will fail to link with "can't find iostream". This patch adds a note
to getting started page. Ok to apply?
Best regards,
Yury Gribov
li
rsmith added a subscriber: rsmith.
rsmith added a comment.
I don't really see this as being specific to modules, nor to static data
members. The natural generalization of this is:
- at the end of a translation unit (including at the end of a module), warn on
any implicit instantiations that are
steven_wu created this revision.
steven_wu added a reviewer: EricWF.
steven_wu added a subscriber: cfe-commits.
--sysroot might cause inconsistency between the header gets used and the
library gets linked on Apple platform. Use -isysroot instead.
http://reviews.llvm.org/D12644
Files:
CMakeList
Author: hans
Date: Fri Sep 4 14:59:39 2015
New Revision: 246864
URL: http://llvm.org/viewvc/llvm-project?rev=246864&view=rev
Log:
Don't allow dllexport/import on static local variables
They might technically have external linkage, but it still doesn't make sense
for the user to try and export su
silvas added a subscriber: silvas.
silvas added a comment.
How expensive is this check?
http://reviews.llvm.org/D12326
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
nwilson added a comment.
Ping
http://reviews.llvm.org/D12435
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tra updated this revision to Diff 34059.
tra marked 2 inline comments as done.
tra added a comment.
Added more test cases to cover constructor/destructor/new/delete.
Refactored code to remove unwanted CUDA functions from Lookup match results.
Disabled HD overloading of H or H destructors to guaran
rsmith added a comment.
I think this is fundamentally the right approach. Anonymous namespaces do
inject a using directive, and we should not forget to pass it to the AST
consumer.
Comment at: lib/CodeGen/CGDebugInfo.cpp:3263-3264
@@ +3262,4 @@
+ const NamespaceDecl *NSDecl =
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
This essentially looks fine. Let me know if you want me to take another look
once you've fixed the `ignorePointerCastsAndParens` bug with derived-to-base
conversions, otherwise go ahead.
===
angelgarcia updated this revision to Diff 34054.
angelgarcia added a comment.
Yes, with SpellingLoc works as well, and it should be better.
http://reviews.llvm.org/D12631
Files:
clang-tidy/modernize/LoopConvertUtils.cpp
clang-tidy/modernize/LoopConvertUtils.h
test/clang-tidy/modernize-loo
Author: aaronballman
Date: Fri Sep 4 13:34:48 2015
New Revision: 246860
URL: http://llvm.org/viewvc/llvm-project?rev=246860&view=rev
Log:
Fixing a bug where hasType(decl()) would fail to match on C code involving
structs or unions.
Modified:
cfe/trunk/include/clang/ASTMatchers/ASTMatchersIn
It turns out that the recordDecl() AST matcher doesn't match
RecordDecl objects; instead, it matches CXXRecordDecl objects. This
is... unfortunate... as it makes writing AST matchers more complicated
because of having to translate between recordDecl()/CXXRecordDecl. It
also makes it impossible to m
klimek added inline comments.
Comment at: clang-tidy/modernize/LoopConvertUtils.cpp:463-470
@@ -462,1 +462,10 @@
+void ForLoopIndexUseVisitor::addUsage(const Usage &U) {
+ SourceLocation Begin = U.Range.getBegin();
+ if (Begin.isMacroID())
+Begin = Context->getSourceManage
angelgarcia updated this revision to Diff 34039.
angelgarcia marked 2 inline comments as done.
angelgarcia added a comment.
Ok! I wasn't sure, and a while would be correct in both cases :)
http://reviews.llvm.org/D12631
Files:
clang-tidy/modernize/LoopConvertUtils.cpp
clang-tidy/modernize/L
klimek added inline comments.
Comment at: clang-tidy/modernize/LoopConvertUtils.cpp:465
@@ +464,3 @@
+ SourceLocation Begin = U.Range.getBegin();
+ while (Begin.isMacroID())
+Begin = Context->getSourceManager().getExpansionLoc(Begin);
There should be no need
Author: ed
Date: Fri Sep 4 11:07:39 2015
New Revision: 246857
URL: http://llvm.org/viewvc/llvm-project?rev=246857&view=rev
Log:
Put ext_implicit_lib_function_decl in ImplicitFunctionDeclare.
If we build with -Werror=implicit-function-declaration, only implicit
function declarations of non-librar
angelgarcia updated this revision to Diff 34035.
angelgarcia added a comment.
I think this solves the duplication problem. And I think is clearer and cheaper
than what I was trying to do before.
http://reviews.llvm.org/D12631
Files:
clang-tidy/modernize/LoopConvertUtils.cpp
clang-tidy/mode
Author: alexfh
Date: Fri Sep 4 10:46:51 2015
New Revision: 246856
URL: http://llvm.org/viewvc/llvm-project?rev=246856&view=rev
Log:
[clang-tidy] Fix llvm-include-order check on Windows.
IncludeDirectives struct used a StringRef that pointed to a stack variable
(SmallString<128> FilenameBuffer fr
Author: alexfh
Date: Fri Sep 4 09:56:57 2015
New Revision: 246854
URL: http://llvm.org/viewvc/llvm-project?rev=246854&view=rev
Log:
[clang-tidy] Fix add_new_check.py.
This fixes the case where the check should be added at the end of
the list of checks in CMakeLists.txt of the corresponding modul
labrinea created this revision.
labrinea added reviewers: rengolin, richard.barton.arm.
labrinea added a subscriber: cfe-commits.
Herald added subscribers: rengolin, aemerson.
Added the remaining macros and corresponding regression tests.
http://reviews.llvm.org/D12633
Files:
lib/Basic/Targets
angelgarcia added a comment.
Anyway, I just found that this needs a few more changes (it still does
duplicated replacementes inside some macros, and I have to find out why), so
don't bother with this for now.
Comment at: clang-tidy/modernize/LoopConvertUtils.cpp:465
@@ +464,3
danielmarjamaki added a subscriber: danielmarjamaki.
danielmarjamaki added a comment.
Overall.. LGTM
Comment at: lib/StaticAnalyzer/Checkers/NullabilityChecker.cpp:78
@@ -78,2 +77,3 @@
+ llvm_unreachable("Unexpected enumeration.");
return "";
}
After llvm_u
alexfh added inline comments.
Comment at: clang-tidy/modernize/LoopConvertUtils.cpp:465
@@ +464,3 @@
+ if (UsageSet.insert(U).second) {
+Usages.push_back(U);
+return true;
Do you need both `Usages` and `UsageSet`?
Comment at: clang-tidy/
Author: abataev
Date: Fri Sep 4 07:55:50 2015
New Revision: 246853
URL: http://llvm.org/viewvc/llvm-project?rev=246853&view=rev
Log:
[OPENMP] Fix false diagnostic on instantiation-dependent exprs for atomic
constructs.
Some of instantiation-dependent expressions could cause false diagnostic to b
Hahnfeld added a comment.
I think this has to be updated for the current trunk...
http://reviews.llvm.org/D9888
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Hahnfeld added a comment.
Needs two small changes to work with current trunk
Comment at: lib/CodeGen/CGStmtOpenMP.cpp:2135-2136
@@ +2134,4 @@
+ const Expr *IfCond = nullptr;
+ if (auto C = S.getSingleClause(OMPC_if)) {
+IfCond = cast(C)->getCondition();
+ }
--
angelgarcia created this revision.
angelgarcia added a reviewer: klimek.
angelgarcia added subscribers: alexfh, cfe-commits.
The InitListExpr subtree is visited twice, this caused the check to do multiple
replacements. Added a set to avoid it.
http://reviews.llvm.org/D12631
Files:
clang-tidy/
Any feedback?
Thanks,
--Serge
2015-08-26 0:26 GMT+06:00 Serge Pavlov :
> sepavloff created this revision.
> sepavloff added a subscriber: cfe-commits.
>
> Instantiation of static class members can be not obvious in some cases.
> Using
> modules can cause problems even more difficult to diagnose.
Author: chapuni
Date: Fri Sep 4 06:49:36 2015
New Revision: 246850
URL: http://llvm.org/viewvc/llvm-project?rev=246850&view=rev
Log:
clang/test/SemaCXX/sourceranges.cpp: Fix silly expressions. Sorry for the
breakage.
Modified:
cfe/trunk/test/SemaCXX/sourceranges.cpp
Modified: cfe/trunk/tes
Author: chapuni
Date: Fri Sep 4 06:40:31 2015
New Revision: 246849
URL: http://llvm.org/viewvc/llvm-project?rev=246849&view=rev
Log:
clang/test/SemaCXX/sourceranges.cpp: Appease win32. Class method is thiscall
for targeting i686-(msvc|mingw32).
Modified:
cfe/trunk/test/SemaCXX/sourceranges.
Author: abataev
Date: Fri Sep 4 06:26:21 2015
New Revision: 246846
URL: http://llvm.org/viewvc/llvm-project?rev=246846&view=rev
Log:
[OPENMP] Fix for http://llvm.org/PR24674: assertion failed and and abort trap
Fix processing of shared variables with reference types in OpenMP constructs.
Previou
MyDeveloperDay retitled this revision from "[Clang-Format] Add
AlwaysBreakBeforeElse Style to avoid cuddled else" to "[Clang-Format] Add
AlwaysBreakBeforeElse and AlwaysBreakBeforeCatch Style to avoid cuddled
else/catch".
MyDeveloperDay updated the summary for this revision.
MyDeveloperDay updat
Author: ogoffart
Date: Fri Sep 4 05:17:10 2015
New Revision: 246844
URL: http://llvm.org/viewvc/llvm-project?rev=246844&view=rev
Log:
Fix the perentheses location when the constructor is called on a class that has
a destructor
Modified:
cfe/trunk/lib/Sema/SemaCast.cpp
cfe/trunk/test/Sem
skalinichev updated this revision to Diff 34019.
skalinichev added a comment.
Rebased.
Yes, now it's ready.
http://reviews.llvm.org/D11797
Files:
test/Index/print-type.c
test/Index/print-type.cpp
tools/libclang/CXType.cpp
Index: tools/libclang/CXType.cpp
On Fri, Aug 28, 2015 at 07:53:40PM +0200, Xan López via cfe-commits wrote:
> This is actually needed, otherwise libc won't be added at all. For
> instance when building libclang.so all the libc symbols won't be
> found, with ld warning about libc being an "implicit dependency".
>
> ((This patch si
On Fri, Aug 28, 2015 at 07:52:17PM +0200, Xan López via cfe-commits wrote:
> There is no __cxa_finalize symbol available on recent Solaris OS
> versions, so we need this flag to make non trivial C++ programs run.
>
> Also stop looking for cxa_finalize.o, since it won't be there.
>
> (This patch s
85 matches
Mail list logo
