On Thu, Sep 5, 2013 at 5:02 PM, Mike Stump wrote:
> On Sep 5, 2013, at 7:42 AM, Joern Rennecke
> wrote:
>> This patch has not been reviewed for more than two weeks:
>> http://gcc.gnu.org/ml/gcc-patches/2013-08/msg00990.html
>
> I'd like a pta type person (or anyone that has an opinion on the cha
On Thu, Sep 5, 2013 at 5:47 PM, Andrew MacLeod wrote:
> On 09/05/2013 09:08 AM, Richard Biener wrote:
>>
>> On Thu, Sep 5, 2013 at 2:57 PM, Andrew MacLeod
>> wrote:
>>>
>>> Now that tree.h is split in two, there are some follow up things that
>>> will
>>> facilitate the deforestation of gimple.
On Thu, Sep 5, 2013 at 11:53 PM, Steven Bosscher wrote:
> On Thu, Sep 5, 2013 at 5:47 PM, Andrew MacLeod wrote:
>> ok, so to dwell on header file cleanup. When creating new header files for
>> say, tree-ssa-ter.h, what other include files should we make assumptions
>> have already been include
On 9/3/13 2:15 PM, Kugan wrote:
> Thanks Richard for reviewing.
>
> On 02/09/13 22:15, Richard Biener wrote:
>> On Wed, Jul 3, 2013 at 2:25 PM, Kugan
>> wrote:
>>> On 17/06/13 18:33, Richard Biener wrote:
On Mon, 17 Jun 2013, Kugan wrote:
+/* Extract the value range of assigned exp
On 9/5/13 11:58 PM, Steven Bosscher wrote:
> On Thu, Sep 5, 2013 at 4:05 PM, Richard Biener wrote:
>>
>> This C++-ifies and moves the control dependence code from tree-ssa-dce.c
>> to cfganal.c as I am about to re-use that code from loop distribution.
>
> I'd recommend re-implementing the control
Mike,
Why does rs6000_init_hard_regno_mode_ok() include #if 0 code?
Because the patch affects e500, has this patch been tested in an e500
configuration?
The cover message did not make it clear that you are merging SFmode
and DFmode patterns using mode iterators. That really should have been
a se
On Thu, 5 Sep 2013, Andrew MacLeod wrote:
> Or are you suggesting that coretypes.h is a file we can assume is available?
Every .c file should start with includes of config.h, system.h and
coretypes.h, in that order, so all other headers can assume they have been
included.
(In principle I think
On Sep 5, 2013, at 2:00 PM, Richard Sandiford
wrote:
> Ping. I should have said that bootstrapping with rtl checking enabled
> is broken as things stand.
Yes, this is fine.
> Richard Sandiford writes:
>> Mike Stump writes:
>>> @@ -643,12 +653,14 @@ equality. */
>>>
On Sep 5, 2013, at 3:50 PM, Jan Hubicka wrote:
> Comitted as obvoius.
>
> Honza
>
> Index: ChangeLog
> ===
> --- ChangeLog (revision 202296)
> +++ ChangeLog (working copy)
> @@ -1,3 +1,6 @@
> +2013-09-05 Jan Hubicka
> +
> + *
On Sep 5, 2013, at 1:52 PM, Caroline Tice wrote:
> Ping? Could somebody please review this for me?
Why? Is one review and approval not enough? :-)
http://gcc.gnu.org/ml/gcc-patches/2013-08/msg01939.html
> On Fri, Aug 30, 2013 at 2:55 PM, Caroline Tice wrote:
>> The attached patch fixes most
Hi,
this very old minor issue is about the kind of diagnostic we want to
produce for something like the existing g++.dg/init/delete1.C:
class C;
void foo(void *p) {
delete [] ((C*)p) ;
}
that is when the array delete is called for a pointer to incomplete
type. Currently we emit an hard
Hi,
this is the patch I commited after testing on x86_64-linux.
Honza
Index: ChangeLog
===
*** ChangeLog (revision 202271)
--- ChangeLog (working copy)
***
*** 1,3
--- 1,9
+ 2013-09-04 Jan Hubicka
+
+
Hi,
ipa-devirt uses mangled names to establish equivalence in between symbols.
I added assert for cloned names, since I convinced myself that otherwise
we can miss matches in between local static symbols. I also convinced
myself we should never rename.
I was wrong on both: declaration merging mak
On Sep 5, 2013, at 12:18 PM, Joern Rennecke wrote:
> OK to apply?
Ok.
On 09/05/2013 06:38 PM, Steven Bosscher wrote:
On Fri, Sep 6, 2013 at 12:22 AM, Andrew MacLeod wrote:
Or are you suggesting that coretypes.h is a file we can assume is available?
that seems like a bit of a slippery slope, but we could pick a few. I
prefer it be explicit myself.
coretypes.h is
On 5 September 2013 13:02:29 Kirill Yukhin wrote:
Hello,
On 04 Sep 20:11, Maxim Kuvyrkov wrote:
> On 4/09/2013, at 7:43 PM, Alexander Ivchenko wrote:
> The patch is OK with definitions of OPTION_GLIBC, OPTION_UCLIBC and
OPTION_BIONIC copied verbatim from gcc/config/l
Checked into main trunk:
Hi,
this testcase breaks with partial inlining because it depends on hand hoisted
loop
header out of the loop nest. Partial inling (correctly) moves the test out of
function
and prevents the analysis to happen.
The whole testcase is very artificial, so I think it is fine to disable partial
inl
On Fri, Sep 6, 2013 at 12:22 AM, Andrew MacLeod wrote:
> Or are you suggesting that coretypes.h is a file we can assume is available?
> that seems like a bit of a slippery slope, but we could pick a few. I
> prefer it be explicit myself.
coretypes.h is available. Why do you think that's a slipper
This fixes a bug where using -fvtable-verify=preinit sometimes causes
an ICE. In particular, when the preinit flag was used, the vtable
verification constructor initialization function was being written to
the assembly file before it was being checked with
cgraph_process_new_functions. This somet
The basic structure we're moving towards for the general form of the FSA
optimization will look something like:
while (something changed)
{
changed = thread_through_empty_blocks ()
if (!threaded_through_block_with_side_effects)
changed |= thread_through_block_with_wide_effects
On Thu, Sep 5, 2013 at 4:05 PM, Richard Biener wrote:
>
> This C++-ifies and moves the control dependence code from tree-ssa-dce.c
> to cfganal.c as I am about to re-use that code from loop distribution.
I'd recommend re-implementing the control dependence code, then. The
current implementation is
On 09/05/2013 05:53 PM, Steven Bosscher wrote:
On Thu, Sep 5, 2013 at 5:47 PM, Andrew MacLeod wrote:
ok, so to dwell on header file cleanup. When creating new header files for
say, tree-ssa-ter.h, what other include files should we make assumptions
have already been included... or should we
On Thu, Sep 5, 2013 at 5:47 PM, Andrew MacLeod wrote:
> ok, so to dwell on header file cleanup. When creating new header files for
> say, tree-ssa-ter.h, what other include files should we make assumptions
> have already been included... or should we make none?
> For instance, the header files
Hi!
I've committed this patch to gomp4 branch to:
1) fix handling of reference based array sections - reference to array
and reference to pointer. The latter actually needs 3 map clauses,
one to map the array section, one to map the pointer to it, and one to
map the reference to the pointer.
2) i
Ping. I should have said that bootstrapping with rtl checking enabled
is broken as things stand.
Thanks,
Richard
Richard Sandiford writes:
> Mike Stump writes:
>> @@ -643,12 +653,14 @@ equality. */
>> __FUNCTION__); \
>> &_rtx->u.hwi
On Thu, 5 Sep 2013, Bernhard Reutner-Fischer wrote:
> uClibc has C99 math support optionally as well as other optional, nonstandard
> feature sets. Your patch does not seem to check (in a cross-compilable
> fashion, of course) if C99 math is supported in libc or not, thus regressing
> on uClibc wi
Richard Biener writes:
>> * wide-int.h no longer includes tm.h.
>>
>> This is done by adding a new MAX_BITS_PER_UNIT to machmode.def,
>> so that the definition of MAX_BITSIZE_MODE_ANY_MODE no longer relies on
>> BITS_PER_UNIT. Although I think we usually assume that BITS_PER_UNIT
>> is a
On 05.09.2013 20:14, Adam Butcher wrote:
On 03.09.2013 22:25, Jason Merrill wrote:
On 09/03/2013 03:50 PM, Adam Butcher wrote:
Problem is that no RTL is set for the incoming parms in the
instantiation of the expansion. It ICEs in gimple_expand_cfg
because 'DECL_RTL_IF_SET
(var)' returns nullp
Ping? Could somebody please review this for me?
-- Caroline Tice
cmt...@google.com
On Fri, Aug 30, 2013 at 2:55 PM, Caroline Tice wrote:
> The attached patch fixes most of the libvtv testsuite tests to run in
> the dejagnu test harness. There are still a few tests that I haven't
> been able to
Ping? Could somebody please review this for me?
-- Caroline Tice
cmt...@google.com
On Wed, Aug 28, 2013 at 2:15 PM, Caroline Tice wrote:
> Discussion on this topic seems to have petered out. I have removed
> references to ENABLE_VTABLE_VERIFY from libvtv, as requested. I
> cannot remove them
Since I posted the port the last time, there have been a number of changes
as well in GCC trunk as in the port, so a new patch set is required to
build the port in trunk.
Sanity-checked by bootstrapping i686-pc-linux-gnu.
OK to apply?
toplevel:
2013-02-25 Simon Cook
Changes to build
On Sep 5, 2013, at 7:42 AM, Joern Rennecke wrote:
> This patch has not been reviewed for more than two weeks:
> http://gcc.gnu.org/ml/gcc-patches/2013-08/msg00990.html
I'd like a pta type person (or anyone that has an opinion on the change) to
review/comment on this… Surely we have at least 1 p
OK to apply?
2013-08-31 Joern Rennecke
Jeremy Bennett
* doc/install.texi (--with-cpu): Mention ARC.
(arc-*-elf32): New paragraph.
(arc-linux-uclibc): Likewise.
* doc/md.texi (Machine Constraints): Add ARC part.
* doc/invoke.texi: (menu): Ad
OK to apply?
FWIW, my colleague Simon Cook is currently working on adding more
target option /
builtin tests.
2013-08-31 Joern Rennecke
Vineet Gupta
* gcc.c-torture/execute/20101011-1.c [__arc__] (DO_TEST): Define as 0.
* testsuite/gcc.target/arc: New directory.
On Tue, Aug 27, 2013 at 3:35 AM, Richard Biener
wrote:
> On Thu, Aug 1, 2013 at 1:34 AM, Easwaran Raman wrote:
>> I have a new patch that supersedes this. The new patch also fixes PR
>> tree-optimization/57393 and PR tree-optimization/58011. Bootstraps and
>> no test regression on x86_64/linux. O
On 03.09.2013 22:25, Jason Merrill wrote:
On 09/03/2013 03:50 PM, Adam Butcher wrote:
Problem is that no RTL is set for the incoming parms in the
instantiation of the expansion. It ICEs in gimple_expand_cfg
because 'DECL_RTL_IF_SET
(var)' returns nullptr for the incoming parms resulting in a f
Quoting Nathan Sidwell :
On 09/05/13 15:50, Joern Rennecke wrote:
Apart from Paolo Carlini's suggestion to improve the wording of the
ChangeLog,
this patch hasn't been reviewed for more than a week:
http://gcc.gnu.org/ml/gcc-patches/2013-08/msg01634.html
I suppose the original subject has
On 09/05/2013 09:08 AM, Richard Biener wrote:
On Thu, Sep 5, 2013 at 2:57 PM, Andrew MacLeod wrote:
Now that tree.h is split in two, there are some follow up things that will
facilitate the deforestation of gimple. I've also thrown in a couple of
structuring issues for good measure.
What are
Apart from Paolo Carlini's suggestion to improve the wording of the ChangeLog,
this patch hasn't been reviewed for more than a week:
http://gcc.gnu.org/ml/gcc-patches/2013-08/msg01634.html
Hi,
This patch fixes vdup_lane_* intrinsics in arm_neon.h to have the
correct lane parameter as opposed to the present '0'.
Tested on aarch64-none-elf. OK for trunk?
Thanks,
Tejas Belagod
ARM.
Changelog:
2013-09-05 Tejas Belagod
gcc/
* config/aarch64/arm_neon.h: Fix all vdup in
> This is all fragile code, so a testcase would be very much appreciated.
A small one is ACATS c45342a (ada/acats/tests/c4/c45342a.ada).
--
Eric Botcazou
On Thu, Sep 5, 2013 at 10:07 AM, Paolo Carlini wrote:
> On 09/04/2013 05:37 AM, Tim Shen wrote:
> Thanks. Apparently there aren't further comments, thus please install it.
Committed. Complete unmatched parentheses in last commit :)
--
Tim Shen
On 09/05/13 15:50, Joern Rennecke wrote:
Apart from Paolo Carlini's suggestion to improve the wording of the ChangeLog,
this patch hasn't been reviewed for more than a week:
http://gcc.gnu.org/ml/gcc-patches/2013-08/msg01634.html
I don't think the issue is whether they should be treated as equi
On 09/05/2013 08:05 AM, Richard Biener wrote:
This C++-ifies and moves the control dependence code from tree-ssa-dce.c
to cfganal.c as I am about to re-use that code from loop distribution.
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Richard.
2013-09-05 Richard Biener
This patch has not been reviewed for more than two weeks:
http://gcc.gnu.org/ml/gcc-patches/2013-08/msg00990.html
On 05/09/13 15:07, Eric Botcazou wrote:
> Hi,
>
> in case the ip register needs to be saved before establishing an APCS frame,
> i.e. when it is used as the static chain register, the prologue tries various
> tricks in the following order:
>
> 1. The last argument register r3.
>
On 09/04/2013 05:37 AM, Tim Shen wrote:
According to this
email(http://gcc.gnu.org/ml/libstdc++/2013-09/msg5.html), I add a
testcase.
Thanks. Apparently there aren't further comments, thus please install it.
Thanks again,
Paolo.
$subject - it's only used from the lto frontend.
LTO bootstrap running on x86_64-unknown-linux-gnu.
Richard.
2013-09-05 Richard Biener
* lto-streamer.h (lto_global_var_decls): Remove.
* Makefile.in (OBJS): Remove lto-symtab.o.
(lto-symtab.o): Remove.
(GTFILES
This C++-ifies and moves the control dependence code from tree-ssa-dce.c
to cfganal.c as I am about to re-use that code from loop distribution.
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Richard.
2013-09-05 Richard Biener
* basic-block.h (class control_dependence
On Tue, 2013-09-03 at 11:08 -0400, Vladimir Makarov wrote:
> The patch is ok for the trunk.
Thanks! Committed as revision 202286 with Jakub's ChangeLog suggestions.
> I'd wait for some time (may be 2 weeks) before committing it for gcc-4.8
> as it might affect other targets (especially code gen
Hi,
in case the ip register needs to be saved before establishing an APCS frame,
i.e. when it is used as the static chain register, the prologue tries various
tricks in the following order:
1. The last argument register r3.
2. A slot on the stack above the frame.
The new extra_objs setting for the ARM was overridden for VxWorks.
Applied on the mainline as obvious.
2013-09-05 Eric Botcazou
* config.gcc (*-*-vxworks*): Do not override an existing extra_objs.
--
Eric BotcazouIndex: config.gcc
==
On 09/05/2013 09:08 AM, Richard Biener wrote:
On Thu, Sep 5, 2013 at 2:57 PM, Andrew MacLeod wrote:
Now that tree.h is split in two, there are some follow up things that will
facilitate the deforestation of gimple. I've also thrown in a couple of
structuring issues for good measure.
What are
On Thu, Sep 5, 2013 at 2:57 PM, Andrew MacLeod wrote:
> Now that tree.h is split in two, there are some follow up things that will
> facilitate the deforestation of gimple. I've also thrown in a couple of
> structuring issues for good measure.
>
> What are your thoughts on these subjects?
Jumpin
Hi,
On Thu, 5 Sep 2013, Richard Biener wrote:
> > (1) whether it's OK for wide_ints to be writable.
> >
> > The interface already exposed the idea of an array of blocks,
> > via get_val() and get_len(). The question here is whether it is OK
> > to also have the corresponding write f
Now that tree.h is split in two, there are some follow up things that
will facilitate the deforestation of gimple. I've also thrown in a
couple of structuring issues for good measure.
What are your thoughts on these subjects?
1 - I think we ought to split out the data structures from gimple
On 05/09/13 10:06, James Greenhalgh wrote:
>
> *PING*
>
> I've rebased this patch on to current trunk, the changes between this and
> the previous version are below:
>
>> @@ -3203,7 +3203,7 @@ (define_insn "*aarch64_ashl_sisd_or_int_
>> (set_attr "simd_type" "simd_shift_imm,simd_shift,*")
>>
This makes the vectorizer no longer produce vector types with
pointer components as it doesn't properly use POINTER_PLUS_EXPR
which then confuses other passes dealing with vectors in a generic way.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Sitting back and waiting fo
This patch slightly improves the code sequence generated when using STRD
instructions instead of PUSH. Rather than emit a stack decrement
operation followed by a sequence of STRDs and a final STR we merge the
stack decrement with the initial store.
Furthermore, we arrange the STR operation so tha
On Wed, 4 Sep 2013 19:48:13, Hans-Peter Nilsson wrote:
> On Tue, 3 Sep 2013, Richard Biener wrote:
>> I think the warning can be completely implemented inside struct-layout.c
>> for example in finish_bitfield_representative (if you pass that the first
>> field
>> in the group, too). Of course that
Hi,
the patch below fixes the first testcase which currently fails to link
because we remove one-too-many references to symbol b and remove it
when creating and inlining speculative call graph edges. It also
removes the reference when speculation is un-done, transparently in
the call graph edge d
On 4 September 2013 15:47, Yufeng Zhang wrote:
> gcc/
>
> * config/aarch64/aarch64-option-extensions.def: Add
> AARCH64_OPT_EXTENSION of 'crc'.
> * config/aarch64/aarch64.h (AARCH64_FL_CRC): New define.
> (AARCH64_ISA_CRC): Ditto.
> * doc/invoke.texi (-marc
Hello,
On 04 Sep 20:11, Maxim Kuvyrkov wrote:
> On 4/09/2013, at 7:43 PM, Alexander Ivchenko wrote:
> The patch is OK with definitions of OPTION_GLIBC, OPTION_UCLIBC and
> OPTION_BIONIC copied verbatim from gcc/config/l
Checked into main trunk: http://gcc.gnu.org/ml/gcc-cvs/2013-09/msg00137.html
On 3 September 2013 12:30, James Greenhalgh wrote:
> 2013-09-03 James Greenhalgh
>
> * config/aarch64/aarch64.md
> (type): Remove frecpe, frecps, frecpx.
> (aarch64_frecp): Move to aarch64-simd.md,
> fix to be a TARGET_SIMD instruction.
> (aarch64_frecp
On Thu, 5 Sep 2013 12:25:08, Richard Biener wrote:
> On Thu, Sep 5, 2013 at 11:43 AM, Bernd Edlinger
> wrote:
>>
>> this would cause an ICE in test case 2629-1.c...
>
> Well, that's easily fixable.
>
>> So removing the pointer of vectors is not an option.
>
> Let me nevertheless try this.
>
>
On 4 September 2013 17:42, Ian Bolton wrote:
> (This patch supercedes this one:
> http://gcc.gnu.org/ml/gcc-patches/2013-07/msg01462.html)
> 2013-09-04 Ian Bolton
>
> gcc/
> * config/aarch64/aarch64.c (aarch64_preferred_reload_class):
> Return NO_REGS for immediate that can't b
On Wed, Sep 4, 2013 at 11:40 AM, David Malcolm wrote:
> On Sat, 2013-08-31 at 19:27 +0200, Richard Biener wrote:
>> David Malcolm wrote:
>> >On Fri, 2013-08-30 at 10:09 +0200, Richard Biener wrote:
>> >> On Thu, Aug 29, 2013 at 9:44 PM, Steven Bosscher
>> > wrote:
>> >> > On Thu, Aug 29, 2013 at
This fixes an ICE I am seeing with local changes - the definition
stmt of an SSA name is never "NULL", but it is a GIMPLE_NOP if
it does not have a definition. Those are the stmts that are not
associated with any basic-block.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard
On Thu, Sep 5, 2013 at 11:43 AM, Bernd Edlinger
wrote:
> On Tue, 3 Sep 2013 10:38:33, Richard Biener wrote:
>> I'd rather go with the simple fix as the issue in forwprop is at least
>> latent. We can
>> improve on the code-gen as followup where I believe handling of
>> POINTER_PLUS_EXPR
>> would n
On Sun, 1 Sep 2013, Richard Sandiford wrote:
> This is an RFC and patch for an alternative way of organising the
> wide-int classes, along the lines I mentioned earlier. The main points
> are below, each with a "headline" and a bit of extra waffle that can be
> skipped if too long:
>
> * As Rich
On Tue, 3 Sep 2013 10:38:33, Richard Biener wrote:
> I'd rather go with the simple fix as the issue in forwprop is at least
> latent. We can
> improve on the code-gen as followup where I believe handling of
> POINTER_PLUS_EXPR
> would need to be added (that we avoid POINTER_PLUS_EXPR for vectors is
On Thu, 5 Sep 2013, Richard Sandiford wrote:
> Kenneth Zadeck writes:
> > It is possible restore the privacy of the original patches but this
> > seems to involve using a private class inside the namespace and using
> > C++ friends.
> >
> > Adding visibility adds extra code
> >
> > namespace wi
On Thu, Sep 5, 2013 at 2:06 AM, Mike Stump wrote:
> On Aug 30, 2013, at 4:22 PM, Diego Novillo wrote:
>> Thanks for the suggestions. I've incorporated them into the patch.
>> It now adds tree-core.h with all the structures, enums, typedefs and
>> some fundamental declarations from tree.h. Every
On Wed, Sep 4, 2013 at 5:40 PM, David Malcolm wrote:
> On Sat, 2013-08-31 at 19:27 +0200, Richard Biener wrote:
>> David Malcolm wrote:
>> >On Fri, 2013-08-30 at 10:09 +0200, Richard Biener wrote:
>> >> On Thu, Aug 29, 2013 at 9:44 PM, Steven Bosscher
>> > wrote:
>> >> > On Thu, Aug 29, 2013 at 6
On 19/08/13 13:46, James Greenhalgh wrote:
>
> Hi,
>
> This patch does two things:
>
> 1. Moves all the "neon_type" attribute values in the ARM backend into
> "type", and removes "neon_type".
>
> 2. Splits type f_2_r and r_2_f to enable them to be used in a similar
> way as neon_mcr et. al.
>
Jakub Jelinek writes:
> That looks wrong, the test was intentionally looking for badly implemented
> vsnprintf, see
> http://www.gnu.org/software/gnulib/manual/html_node/snprintf.html
> "This function does not return a byte count as specified in C99 on some
> platforms: HP-UX 11, IRIX 6.5, OSF/1
On Wed, Sep 04, 2013 at 08:54:47PM +0200, Jakub Jelinek wrote:
> Any issues you can spot in the patch? If not, I'll commit it tomorrow, so
> that I can continue poking in these areas without too much pending stuff.
Committed now.
> Known issues that don't affect host fallback, but will be a prob
> On 09/04/2013 10:49 AM, Bernd Schmidt wrote:
> >On 09/04/2013 06:04 PM, Jan Hubicka wrote:
> >>this is third fallout of my change to remove DECL_ARGUMENTS/DECL_RESULT for
> >>functions w/o
> >>bodies I did not really anticipate.
> >[...]
> >>I would like to basically ask if it seems to make sens
Ian Lance Taylor a écrit:
> On Wed, Sep 4, 2013 at 1:46 AM, Dodji Seketeli wrote:
>>
>> +/* In a FUNCTION_DECL, nonzero means a built in function of a
>> + standard library or more generally a built in function that is
>> + recognized by optimizers and expanders.
>> +
>> + Note that it is
On 08/30/2013 05:50 PM, Joseph S. Myers wrote:
> On Fri, 30 Aug 2013, Christian Bruel wrote:
>
>> So to cross build a target library |
>> --with-build-sysroot=|dir looks appropriate to specify the alternative
>> host root path.
>> but
>> --with-sysroot looks not appropriate because it changes the
Kenneth Zadeck writes:
> It is possible restore the privacy of the original patches but this
> seems to involve using a private class inside the namespace and using
> C++ friends.
>
> Adding visibility adds extra code
>
> namespace wi {
> class impl {
> private:
>
81 matches
Mail list logo
