vmiklos added a comment.
As mentioned earlier, I have no problem with merging rename-at and rename-all.
https://reviews.llvm.org/D24567
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
Hahnfeld created this revision.
Hahnfeld added a reviewer: t.p.northover.
Hahnfeld added a subscriber: cfe-commits.
Until someone rewrites the stdlib logic for Darwin so that we don't need
to pass down the -stdlib argument to cc1.
https://reviews.llvm.org/D24601
Files:
test/Driver/darwin-stdli
EricWF added a comment.
I should mention that before this patch a GCC compiled dylib had 62 additional
symbols and was missing 105 symbols. Now there are only 20 missing symbols and
62 additional.
https://reviews.llvm.org/D24600
___
cfe-commits ma
ioeric requested changes to this revision.
This revision now requires changes to proceed.
Comment at: clang-tidy/ClangTidy.cpp:25
@@ -24,2 +24,3 @@
#include "clang/ASTMatchers/ASTMatchFinder.h"
+#include "clang/Format/Format.h"
#include "clang/Frontend/ASTConsumers.h"
--
EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.
This approach looks great. I don't think we need to split things up into
separate files. This is exactly how `test_macros.h` should be used.
Feel free to convert as many tests as possible. I'v
Author: olista01
Date: Thu Sep 15 03:55:41 2016
New Revision: 281596
URL: http://llvm.org/viewvc/llvm-project?rev=281596&view=rev
Log:
[ARM] ARM-specific attributes should be accepted for big-endian
The ARM-specific C attributes (currently just interrupt) need to check
for both the big- and littl
This revision was automatically updated to reflect the committed changes.
Closed by commit rL281596: [ARM] ARM-specific attributes should be accepted for
big-endian (authored by olista01).
Changed prior to commit:
https://reviews.llvm.org/D24245?vs=70364&id=71485#toc
Repository:
rL LLVM
htt
EricWF created this revision.
EricWF added a reviewer: mclow.lists.
EricWF added a subscriber: cfe-commits.
Herald added subscribers: mgorny, beanz.
This patch fixes a number of problems with the visibility macros across GCC (on
Unix) and Windows (DLL import/export semantics). All of the visibili
kfunk added a comment.
@other LLVM devs: Please review so we can finally ship this?
Comment at: tools/libclang/CIndex.cpp:4311
@@ -4311,1 +4310,3 @@
+ C.kind == CXCursor_ConversionFunction ||
+ C.kind == CXCursor_FunctionDecl) {
if (pieceIndex > 0)
---
EricWF updated this revision to Diff 71486.
EricWF added a comment.
Fix obvious errors in docs.
https://reviews.llvm.org/D24602
Files:
CMakeLists.txt
docs/DesignDocs/VisibilityMacros.rst
docs/index.rst
include/__config
include/__locale
include/__string
include/experimental/filesys
EricWF added inline comments.
Comment at: include/__config:605
@@ +604,3 @@
+# if __has_attribute(__type_visibility__)
+#define _LIBCPP_EXTERN_TEMPLATE_TYPE_VIS __attribute__
((__visibility__("default")))
+# else
Checking `__type_visibility__` and using `__
rmaprath added inline comments.
Comment at: test/std/re/re.alg/re.alg.search/grep.pass.cpp:25
@@ -25,2 +24,3 @@
+#ifndef TEST_HAS_NO_EXCEPTIONS
extern "C" void LLVMFuzzerTestOneInput(const char *data)
EricWF wrote:
> I prefer putting the `#if` on the inside of
sdardis added a comment.
Ping.
https://reviews.llvm.org/D21070
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: kuba.brecka
Date: Thu Sep 15 06:04:53 2016
New Revision: 281603
URL: http://llvm.org/viewvc/llvm-project?rev=281603&view=rev
Log:
[libcxx] Allow sanitizing libcxx with ASan+UBSan simultaneously
Allow building with LLVM_USE_SANITIZER=“Address;Undefined” (and
“Undefined;Address”).
Differe
ioeric added a comment.
Ping
https://reviews.llvm.org/D24380
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ioeric updated this revision to Diff 71495.
ioeric added a comment.
- Update comment in `getAffectedRanges`.
https://reviews.llvm.org/D24606
Files:
include/clang/Tooling/Core/Replacement.h
lib/Tooling/Core/Replacement.cpp
unittests/Tooling/RefactoringTest.cpp
Index: unittests/Tooling/Ref
hokein added inline comments.
Comment at: clang-tidy/ClangTidy.cpp:206
@@ +205,3 @@
+StringRef Code = Buffer.get()->getBuffer();
+format::FormatStyle Style = format::getLLVMStyle();
+llvm::Expected CleanReplacements =
ioeric wrote:
> Add a
milianw accepted this revision.
milianw added a comment.
This revision is now accepted and ready to land.
agreed, lgtm but someone else must accept this upstream
https://reviews.llvm.org/D18462
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
ioeric added a comment.
PIng
https://reviews.llvm.org/D24183
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mstorsjo created this revision.
mstorsjo added a reviewer: compnerd.
mstorsjo added a subscriber: cfe-commits.
Herald added subscribers: samparker, rengolin, aemerson.
On ARM, there are multiple versions of each of the intrinsics, with
acquire/relaxed/release barrier semantics.
The 64 bit versio
vbyakovlcl updated this revision to Diff 71498.
https://reviews.llvm.org/D24467
Files:
llvm/tools/clang/lib/Sema/SemaExpr.cpp
llvm/tools/clang/test/CodeGen/vecshift.c
llvm/tools/clang/test/Sema/vecshift.c
Index: llvm/tools/clang/lib/Sema/SemaExpr.cpp
===
Author: aaronballman
Date: Thu Sep 15 09:01:10 2016
New Revision: 281609
URL: http://llvm.org/viewvc/llvm-project?rev=281609&view=rev
Log:
Silence false positive diagnostics regarding passing an object of enumeration
type to va_start(). The underlying type for an enumeration in C is either char,
aaron.ballman closed this revision.
aaron.ballman added a comment.
Commit in r281609
https://reviews.llvm.org/D23921
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hokein added inline comments.
Comment at: change-namespace/ChangeNamespace.cpp:200
@@ +199,3 @@
+ while (!NsSplitted.empty()) {
+// FIXME: consider code style for comments.
+Code = ("namespace " + NsSplitted.back() + " {\n" + Code +
Doesn't `formatAndAppl
Author: sdardis
Date: Thu Sep 15 09:01:55 2016
New Revision: 281610
URL: http://llvm.org/viewvc/llvm-project?rev=281610&view=rev
Log:
[mips][ias] Enable IAS by default for N64 on Debian mips64el.
Unfortunately we can't enable it for all N64 because it is not yet possible to
distinguish N32 from N
sdardis closed this revision.
sdardis added a comment.
Committed as https://reviews.llvm.org/rL281610.
https://reviews.llvm.org/D22679
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: aaronballman
Date: Thu Sep 15 09:12:33 2016
New Revision: 281612
URL: http://llvm.org/viewvc/llvm-project?rev=281612&view=rev
Log:
Reverting r281609; it caused some build bots to break.
http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fast/builds/20061/steps/test/lo
hokein updated this revision to Diff 71506.
hokein marked 2 inline comments as done.
hokein added a comment.
Herald added a subscriber: mgorny.
Address review comments.
https://reviews.llvm.org/D24243
Files:
CMakeLists.txt
clang-move/CMakeLists.txt
clang-move/ClangMove.cpp
clang-move/Cl
hokein added inline comments.
Comment at: clang-move/ClangMove.cpp:104
@@ +103,3 @@
+ std::reverse(Namespaces.begin(), Namespaces.end());
+ return Namespaces;
+}
Aha, I see. I misused the `findLocationAfterToken` previously.
Comment at: clang-m
tavianator updated this revision to Diff 71508.
tavianator added a comment.
Herald added subscribers: mgorny, beanz.
Integrated @EricWF's expanded test case, and avoid an unneeded
pthread_setspecific() call if the last thread_local's destructor initializes a
new thread_local.
https://reviews.l
tavianator marked 5 inline comments as done.
Comment at: src/cxa_thread_atexit.cpp:70
@@ +69,3 @@
+while (auto head = dtors) {
+ dtors = head->next;
+ head->dtor(head->obj);
EricWF wrote:
> tavianator wrote:
> > EricWF wrote:
> > > tavianator wrote:
NoQ updated this revision to Diff 71510.
NoQ marked an inline comment as done.
NoQ added a comment.
Herald added subscribers: mgorny, beanz.
Added a huge amount of macros in order to improve readability of function specs.
Other inline comments should have been addressed before.
https://reviews.l
Author: dergachev
Date: Thu Sep 15 11:25:42 2016
New Revision: 281625
URL: http://llvm.org/viewvc/llvm-project?rev=281625&view=rev
Log:
[analyzer] Fix HTMLRewriter style sheets to support non-webkit browsers.
This fixes rounded corners and shadows of analyzer diagnostic pieces
in browsers such as
This revision was automatically updated to reflect the committed changes.
Closed by commit rL281625: [analyzer] Fix HTMLRewriter style sheets to support
non-webkit browsers. (authored by dergachev).
Changed prior to commit:
https://reviews.llvm.org/D23272?vs=67182&id=71517#toc
Repository:
rL
mclow.lists added a comment.
Any reason we shouldn't just revert r280944, wait for the LLVM bug to be fixed,
and then re-apply it?
https://reviews.llvm.org/D24599
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin
davidsh created this revision.
davidsh added reviewers: carlo.bertolli, arpith-jacob, kkwli0, sfantao, ABataev.
davidsh added a subscriber: cfe-commits.
#pragma omp for
{
for(...)
}
This is technically not allowed by the standard, gcc doesnt allow such code.
https://reviews.llvm.org/D24615
Fi
Author: aaronballman
Date: Thu Sep 15 13:07:51 2016
New Revision: 281632
URL: http://llvm.org/viewvc/llvm-project?rev=281632&view=rev
Log:
Reapply: Silence false positive diagnostics regarding passing an object of
enumeration type to va_start().
The underlying type for an enumeration in C is eit
thakis added a comment.
Works for me if rnk likes it :-)
(We could bikeshed on if this should be one of the -Wmicrosoft warnings, but
the warning can't be both in -Wall and -Wmicrosoft, so let's don't).
I do have a question about the test (the first commit below):
Comment at:
loladiro updated this revision to Diff 71528.
loladiro added a comment.
Add private copy of forward (__forward) that is constexpr even in C++11 mode,
use test suggested by @rsmith
Repository:
rL LLVM
https://reviews.llvm.org/D24372
Files:
include/memory
test/std/utilities/memory/unique
rnk added a comment.
In https://reviews.llvm.org/D24289#543874, @thakis wrote:
> Works for me if rnk likes it :-)
Yep, looks good.
> (We could bikeshed on if this should be one of the -Wmicrosoft warnings, but
> the warning can't be both in -Wall and -Wmicrosoft, so let's don't).
I also don
aaron.ballman accepted this revision.
aaron.ballman added a reviewer: aaron.ballman.
aaron.ballman added a comment.
LGTM as well, thank you!
https://reviews.llvm.org/D24289
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.or
Author: smeenai
Date: Thu Sep 15 13:36:13 2016
New Revision: 281641
URL: http://llvm.org/viewvc/llvm-project?rev=281641&view=rev
Log:
[libc++] Avoid include in locale_win32.h
When `_LIBCPP_NO_EXCEPTIONS` is defined, we end up with compile errors
when targeting MSVCRT:
* Code includes ``
* `` in
This revision was automatically updated to reflect the committed changes.
Closed by commit rL281641: [libc++] Avoid include in locale_win32.h
(authored by smeenai).
Changed prior to commit:
https://reviews.llvm.org/D24374?vs=70899&id=71530#toc
Repository:
rL LLVM
https://reviews.llvm.org/D
hiraditya added a comment.
@EricWF, since inline is only a hint, the compiler would not inline in many
cases, it might give the inliner a little bit of push to inline. When we were
working on this patch, adding inline wasn't enough and hence we added the
_LIBCPP_INLINE_VISIBILITY flag. The comp
EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.
LGTM. Thanks for the patch.
Comment at: src/cxa_thread_atexit.cpp:70
@@ +69,3 @@
+
+ void run_dtors(void*) {
+while (auto head = dtors) {
Na I thought I
EricWF added a comment.
In https://reviews.llvm.org/D24599#543849, @mclow.lists wrote:
> Any reason we shouldn't just revert r280944, wait for the LLVM bug to be
> fixed, and then re-apply it?
I would like to put some time between fixing the Clang bug and re-introducing
the reproducer into li
ahatanak added a comment.
LGTM with a nit in test case.
Comment at: llvm/tools/clang/test/CodeGen/vecshift.c:43
@@ +42,3 @@
+ vi8 = 1 << vi8;
+// CHECK: [[t2:%.+]] = load <8 x i32>, <8 x i32>* @vi8, align 32
+// CHECK: shl <8 x i32> , [[t2]]
This test fails on
Author: hans
Date: Thu Sep 15 14:44:49 2016
New Revision: 281648
URL: http://llvm.org/viewvc/llvm-project?rev=281648&view=rev
Log:
clang-format VS plugin: upgrade the project files to VS2015
The plugin itself runs on previous VS versions, but this enables
it to be built with VS2015.
Modified:
vbyakovlcl added inline comments.
Comment at: llvm/tools/clang/test/CodeGen/vecshift.c:43
@@ +42,3 @@
+ vi8 = 1 << vi8;
+// CHECK: [[t2:%.+]] = load <8 x i32>, <8 x i32>* @vi8, align 32
+// CHECK: shl <8 x i32> , [[t2]]
ahatanak wrote:
> This test fails on my mac
vbyakovlcl updated this revision to Diff 71545.
https://reviews.llvm.org/D24467
Files:
llvm/tools/clang/lib/Sema/SemaExpr.cpp
llvm/tools/clang/test/CodeGen/vecshift.c
llvm/tools/clang/test/Sema/vecshift.c
Index: llvm/tools/clang/lib/Sema/SemaExpr.cpp
===
- cfe-dev, + cfe-commits
Hi Tobias,
This is a great start. Thanks for the patch!
> Not sure I know how to use the analyzer config flags yet. My patch
> currently also still causes a segfault on Analysis/retain-release.m
The segfault is because the diagnostics machinery expects a declaration wit
yaxunl created this revision.
yaxunl added reviewers: Anastasia, bader.
yaxunl added subscribers: cfe-commits, nhaustov, rampitec.
Herald added a subscriber: yaxunl.
https://reviews.llvm.org/D24626
Files:
include/clang/Basic/DiagnosticSemaKinds.td
lib/Sema/SemaExpr.cpp
test/SemaOpenCL/half.
andriigrynenko created this revision.
andriigrynenko added reviewers: kcc, blastrock, dvyukov, filcab.
andriigrynenko added a subscriber: cfe-commits.
andriigrynenko set the repository for this revision to rL LLVM.
Herald added a subscriber: kubabrecka.
This patch extends __sanitizer_finish_switch
aaron.ballman added a comment.
Thank you for working on this check! A few comments:
The patch is missing Sema tests for the attribute (that it only applies to
declarations you expect, accepts no args, etc).
Have you considered making this a type attribute on the return type of the
function rat
blastrock added a comment.
Seems fine to me, but I think you forgot to update the tests :)
Repository:
rL LLVM
https://reviews.llvm.org/D24628
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/
aaron.ballman added inline comments.
Comment at: lib/Sema/SemaDeclAttr.cpp:4967
@@ +4966,3 @@
+
+ D->addAttr(::new (S.Context)
+ AMDGPUFlatWorkGroupSizeAttr(Attr.getLoc(), S.Context, Min, Max,
Is it okay to supply `0, 0` as the min, max arguments?
==
EricWF added a comment.
Thanks for working on this. That static initialization bug is ugly.
My only concern with this patch is that adding `constexpr` to unconstrained
constructors is going to result compile errors caused by Clang's eager
instantiation.
Also, unlike @mclow.lists, I think we sh
twoh updated this revision to Diff 71560.
twoh added a comment.
Tests added
https://reviews.llvm.org/D23765
Files:
lib/Sema/SemaExprCXX.cpp
test/SemaCXX/crash-has-nothrow-constructor.cpp
test/SemaCXX/crash-has-nothrow-copy.cpp
Index: test/SemaCXX/crash-has-nothrow-copy.cpp
==
bkramer accepted this revision.
bkramer added a comment.
looks good from my side.
https://reviews.llvm.org/D18462
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rsmith added inline comments.
Comment at: lib/Sema/SemaExprCXX.cpp:4227
@@ -4226,1 +4226,3 @@
continue;
+// Using(Shadow)Decl itself is not a constructor
+if (isa(ND) || isa(ND))
This isn't really right: a `UsingShadowDecl` whose underly
EricWF resigned from this revision.
EricWF removed a reviewer: EricWF.
EricWF added a comment.
After talking with @rmaprath we have agreed to go in a different direction.
Resigning as reviewer to keep my queue clean.
https://reviews.llvm.org/D23926
___
Author: kcc
Date: Thu Sep 15 17:11:08 2016
New Revision: 281665
URL: http://llvm.org/viewvc/llvm-project?rev=281665&view=rev
Log:
[sanitizer-coverage] make trace-pc-guard and indirect-call work together
Modified:
cfe/trunk/docs/SanitizerCoverage.rst
Modified: cfe/trunk/docs/SanitizerCoverage
Author: dlj
Date: Thu Sep 15 17:12:26 2016
New Revision: 281666
URL: http://llvm.org/viewvc/llvm-project?rev=281666&view=rev
Log:
Simplify Clang's version number configuration in CMake.
Currently, the Clang version is computed as follows:
1. LLVM defines major, minor, and patch versions, all st
This revision was automatically updated to reflect the committed changes.
Closed by commit rL281669: [Sema] Allow shifting a scalar operand by a vector
operand. (authored by ahatanak).
Changed prior to commit:
https://reviews.llvm.org/D24467?vs=71545&id=71569#toc
Repository:
rL LLVM
https:/
Author: ahatanak
Date: Thu Sep 15 17:19:25 2016
New Revision: 281669
URL: http://llvm.org/viewvc/llvm-project?rev=281669&view=rev
Log:
[Sema] Allow shifting a scalar operand by a vector operand.
r278501 inadvertently introduced a bug in which it disallowed shifting
scalar operands by vector opera
EricWF abandoned this revision.
EricWF added a comment.
Abandoning. All issues have been addressed by other commits.
Comment at: include/memory:5452-5458
@@ +5451,9 @@
+ {
+return static_cast<_To>(_FromPtrTraits::pointer_to(__e));
+ }
+
+ _LIBCPP_INLINE_VISIBILITY
+ stat
EricWF updated this revision to Diff 71570.
EricWF added a comment.
I'm going to go ahead and commit this w/o review because it's blocking more
important changes. Post-commit comments on the documentation are appreciated.
https://reviews.llvm.org/D24602
Files:
CMakeLists.txt
docs/DesignDoc
Author: ericwf
Date: Thu Sep 15 17:27:07 2016
New Revision: 281673
URL: http://llvm.org/viewvc/llvm-project?rev=281673&view=rev
Log:
[libc++] Fix and document visibility attributes for Clang, GCC and Windows.
Summary:
This patch fixes a number of problems with the visibility macros across GCC (on
dcoughlin added a comment.
Thanks for adding the macros. I've provided some feedback inline.
I think a good rule of thumb for readability is: suppose you are a maintainer
and need to add a summary for a new function. Can you copy the the summary for
an existing function and figure out what each
erik.pilkington created this revision.
erik.pilkington added reviewers: faisalv, rsmith.
erik.pilkington added a subscriber: cfe-commits.
Previously, clang emitted no diagnostic for the following:
```
auto f() {
int loc;
return [&] { return loc; };
}
```
The problem being that this returns a d
erik.pilkington added a comment.
Ping, sorry for the delay.
https://reviews.llvm.org/D22053
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Well, on my machine $(SDKToolsDir) doesn't work :-( I suspect the file
will need manual tweaking by whoever is trying to build the plugin.
Anyway, I've updated the solution to build with VS2015 in r281648 and
confirmed that it can still be used with older VS versions too.
Cheers,
Hans
On Thu, Au
You may need to install the Visual Studio SDK. Did you do that when you
initially installed VS 2015?
On Thu, Sep 15, 2016 at 4:15 PM Hans Wennborg wrote:
> Well, on my machine $(SDKToolsDir) doesn't work :-( I suspect the file
> will need manual tweaking by whoever is trying to build the plugin
When I first opened the solution in VS it prompted me to install it and I did.
On Thu, Sep 15, 2016 at 4:17 PM, Zachary Turner wrote:
> You may need to install the Visual Studio SDK. Did you do that when you
> initially installed VS 2015?
>
> On Thu, Sep 15, 2016 at 4:15 PM Hans Wennborg wrote:
Strange. FWIW you can dump all the variables that are present in your
environment. You need to go to Tools -> Options -> Projects and Solutions
-> Build and Run and choose either Normal, Detailed, or Diagnostic for the
MSBuild project build output verbosity. Then in the output window you will
ge
ahatanak abandoned this revision.
ahatanak added a comment.
The crash I was trying to fix was fixed in https://reviews.llvm.org/D23643.
https://reviews.llvm.org/D14471
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi
jlebar created this revision.
jlebar added a reviewer: kcc.
jlebar added subscribers: jhen, tra, cfe-commits.
Sanitizers aren't supported on NVPTX -- don't try to run them.
This lets you e.g. pass -fsanitize=address and get asan on your host
code.
https://reviews.llvm.org/D24640
Files:
clang/
kcc accepted this revision.
kcc added a comment.
This revision is now accepted and ready to land.
LGTM
https://reviews.llvm.org/D24640
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: jlebar
Date: Thu Sep 15 18:44:13 2016
New Revision: 281680
URL: http://llvm.org/viewvc/llvm-project?rev=281680&view=rev
Log:
[CUDA] Don't try to run sanitizers on NVPTX.
Summary:
Sanitizers aren't supported on NVPTX -- don't try to run them.
This lets you e.g. pass -fsanitize=address and
This revision was automatically updated to reflect the committed changes.
Closed by commit rL281680: [CUDA] Don't try to run sanitizers on NVPTX.
(authored by jlebar).
Changed prior to commit:
https://reviews.llvm.org/D24640?vs=71572&id=71573#toc
Repository:
rL LLVM
https://reviews.llvm.org
EricWF updated this revision to Diff 71574.
EricWF added a comment.
- Update the patch based on r281673.
- Add doc for `_LIBCPP_EXTERN_TEMPLATE_INLINE_VISIBILITY`
- Move all uses of `_LIBCPP_EXTERN_TEMPLATE_INLINE_VISIBILITY` to the first
declaration in order to support `__attribute__((internal_l
EricWF accepted this revision.
EricWF added a reviewer: EricWF.
EricWF added a comment.
This revision is now accepted and ready to land.
Accepting to commit. Post commit review is very welcome (but nobody likes
looking at visibility patches).
https://reviews.llvm.org/D24600
_
Author: ericwf
Date: Thu Sep 15 19:00:48 2016
New Revision: 281681
URL: http://llvm.org/viewvc/llvm-project?rev=281681&view=rev
Log:
[libc++] Add _LIBCPP_EXTERN_TEMPLATE_INLINE_VISIBILITY to support GCC ABI
compatibility
Summary:
GCC and Clang handle visibility attributes on the out-of-line defi
This revision was automatically updated to reflect the committed changes.
Closed by commit rL281682: [CodeGen][ObjC] Block captures should inherit the
type of the captured (authored by ahatanak).
Changed prior to commit:
https://reviews.llvm.org/D21104?vs=64199&id=71575#toc
Repository:
rL LL
Author: ahatanak
Date: Thu Sep 15 19:02:06 2016
New Revision: 281682
URL: http://llvm.org/viewvc/llvm-project?rev=281682&view=rev
Log:
[CodeGen][ObjC] Block captures should inherit the type of the captured
field in the enclosing lambda or block.
This patch fixes a bug in code-gen where it uses th
Author: ericwf
Date: Thu Sep 15 19:07:16 2016
New Revision: 281683
URL: http://llvm.org/viewvc/llvm-project?rev=281683&view=rev
Log:
Move inline attributes in filesystem to first declaration
Modified:
libcxx/trunk/include/experimental/filesystem
Modified: libcxx/trunk/include/experimental/fi
Author: ericwf
Date: Thu Sep 15 19:13:55 2016
New Revision: 281684
URL: http://llvm.org/viewvc/llvm-project?rev=281684&view=rev
Log:
Use _LIBCPP_EXTERN_TEMPLATE_INLINE_VISIBILITY in valarray to support
attribute((internal_linkage)).
The externally instantiated member functions must be declared u
EricWF created this revision.
EricWF added reviewers: mclow.lists, eugenis.
EricWF added subscribers: cfe-commits, eugenis.
This patch has been a long time coming (Thanks @eugenis). It changes
`_LIBCPP_INLINE_VISIBILITY` to use `__attribute__((internal_linkage))` instead
of `__attribute__((visib
EricWF updated this revision to Diff 71579.
EricWF added a comment.
Revert accidental change.
https://reviews.llvm.org/D24642
Files:
include/__config
src/string.cpp
Index: src/string.cpp
===
--- src/string.cpp
+++ src/string.c
meikeb updated this revision to Diff 71581.
meikeb added a comment.
Rebase to current commit.
https://reviews.llvm.org/D24584
Files:
lib/CodeGen/CGBlocks.cpp
lib/CodeGen/CGBlocks.h
lib/Sema/SemaChecking.cpp
test/CodeGenObjCXX/lambda-expressions.mm
test/Sema/format-strings.c
Index: te
meikeb updated this revision to Diff 71583.
meikeb added a comment.
Try to drop randomly uploaded commit.
https://reviews.llvm.org/D24584
Files:
lib/Sema/SemaChecking.cpp
test/Sema/format-strings.c
Index: test/Sema/format-strings.c
==
meikeb updated this revision to Diff 71584.
meikeb added a comment.
Rebase
https://reviews.llvm.org/D24584
Files:
lib/CodeGen/CGBlocks.cpp
lib/CodeGen/CGBlocks.h
lib/Sema/SemaChecking.cpp
test/CodeGenObjCXX/lambda-expressions.mm
test/Sema/format-strings.c
Index: test/Sema/format-stri
meikeb updated this revision to Diff 71585.
meikeb added a comment.
.
https://reviews.llvm.org/D24584
Files:
lib/Sema/SemaChecking.cpp
test/Sema/format-strings.c
Index: test/Sema/format-strings.c
===
--- test/Sema/format-strin
Author: srhines
Date: Thu Sep 15 20:07:04 2016
New Revision: 281686
URL: http://llvm.org/viewvc/llvm-project?rev=281686&view=rev
Log:
Do not warn about format strings that are indexed string literals.
Summary:
The warning for a format string not being a string literal and therefore
being potentia
This revision was automatically updated to reflect the committed changes.
Closed by commit rL281686: Do not warn about format strings that are indexed
string literals. (authored by srhines).
Changed prior to commit:
https://reviews.llvm.org/D24584?vs=71585&id=71586#toc
Repository:
rL LLVM
h
eugenis accepted this revision.
eugenis added a comment.
This revision is now accepted and ready to land.
Looks great.
Thank you for seeing it through!
https://reviews.llvm.org/D24642
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://li
rsmith added a comment.
> But not here, because we would have to verify that the pointer in lam wasn't
> mutated in a previous call of the lambda:
Isn't that guaranteed, because the lambda is not marked `mutable`?
Comment at: lib/Sema/SemaChecking.cpp:6594
@@ -6590,1 +6593,3
Author: ericwf
Date: Thu Sep 15 21:09:26 2016
New Revision: 281691
URL: http://llvm.org/viewvc/llvm-project?rev=281691&view=rev
Log:
Move _LIBCPP_INLINE_VISIBILITY to first declaration in sstream.
Modified:
libcxx/trunk/include/sstream
Modified: libcxx/trunk/include/sstream
URL:
http://llvm
Author: ericwf
Date: Thu Sep 15 21:16:23 2016
New Revision: 281692
URL: http://llvm.org/viewvc/llvm-project?rev=281692&view=rev
Log:
Move _LIBCPP_INLINE_VISIBILITY to first declaration in
Modified:
libcxx/trunk/include/experimental/propagate_const
Modified: libcxx/trunk/include/experimental
EricWF updated this revision to Diff 71590.
EricWF added a comment.
- Add `always_inline` to _LIBCPP_INLINE_VISIBILITY. In future I think only
_LIBCPP_ALWAYS_INLINE should actually apply `always_inline`, but I'll make that
change separately.
- Change `_LIBCPP_ALWAYS_INLINE` to use `internal_link
1 - 100 of 116 matches
Mail list logo