Author: ericwf
Date: Sat Sep 5 02:20:53 2015
New Revision: 246915
URL: http://llvm.org/viewvc/llvm-project?rev=246915&view=rev
Log:
Try and fix links again. Seems to be a sphinx version issue.
Modified:
libcxx/trunk/docs/index.rst
Modified: libcxx/trunk/docs/index.rst
URL:
http://llvm.org/
klimek added a comment.
Note that I think we should add a regression test that will work with the
spelling loc but not with the expansion loc.
Something like:
#define A \
\
A
http://reviews.llvm.org/D12631
___
cfe-commits mailing list
cfe-c
espositofulvio updated the summary for this revision.
espositofulvio updated this revision to Diff 34104.
espositofulvio added a comment.
- Addressed possible ABI breaks
- Reverted to not using a __config_file as @jroelofs has two separate patch for
that
Repository:
rL LLVM
http://reviews.ll
randomcppprogrammer added a reviewer: aaron.ballman.
randomcppprogrammer updated this revision to Diff 34111.
randomcppprogrammer added a comment.
Updated check according to comments given by Aaron Ballman. Most notable
change: added optional check for throwing anonmyous temporaries.
http://rev
Author: compnerd
Date: Sat Sep 5 13:53:43 2015
New Revision: 246931
URL: http://llvm.org/viewvc/llvm-project?rev=246931&view=rev
Log:
Index: expose visibility attribute
Expose the previously unexposed visibility attribute via the python and C
bindings.
Added:
cfe/trunk/test/Index/visibility
On Fri, Sep 04, 2015 at 09:28:39PM +, Eric Fiselier via cfe-commits wrote:
> 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 poten
On Fri, Sep 4, 2015 at 1:11 AM, Xan López via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> 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+
On Fri, Sep 4, 2015 at 1:12 AM, Xan López via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> 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 sym
danalbert added a comment.
Android has posix_memalign too.
http://reviews.llvm.org/D12512
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Android has posix_memalign too.
On Sep 5, 2015 12:00, "Joerg Sonnenberger" wrote:
> On Fri, Sep 04, 2015 at 09:28:39PM +, Eric Fiselier via cfe-commits
> wrote:
> > EricWF added a comment.
> >
> > In http://reviews.llvm.org/D12512#237175, @joerg wrote:
> >
> > > Please don't commit this as is
On Sat, Sep 05, 2015 at 07:29:43PM +, Dan Albert via cfe-commits wrote:
> danalbert added a comment.
>
> Android has posix_memalign too.
Yes, but people seem to be reluctant to use it. See the linked review.
Joerg
___
cfe-commits mailing list
cfe-c
piotrdz added a comment.
Now that I fixed all review issues, I think this version would be acceptable
for commit. @alexfh: do you agree?
There are of course two outstanding issues marked with TODO and FIXME comments,
but these are areas of improvement. I would like to first discuss the best
sol
piotrdz updated this revision to Diff 34113.
piotrdz marked 6 inline comments as done.
piotrdz added a comment.
I addressed all latest review issues.
http://reviews.llvm.org/D12462
Files:
clang-tidy/readability/CMakeLists.txt
clang-tidy/readability/InconsistentDeclarationParameterNameCheck.
jtouton added a comment.
Added cfe-commits.
http://reviews.llvm.org/D12664
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
14 matches
Mail list logo
