* [PATCH] Cleanup last_location and update input_location in ipa_prop
@ 2012-11-06 1:20 Dehao Chen
2012-11-08 17:39 ` Dehao Chen
0 siblings, 1 reply; 10+ messages in thread
From: Dehao Chen @ 2012-11-06 1:20 UTC (permalink / raw)
To: GCC Patches
[-- Attachment #1: Type: text/plain, Size: 482 bytes --]
Hi,
This is a patch to do some obvious cleanup and setting correct
input_location in ipa_prop (because it invokes gimplification
routines).
Bootstrapped and passed gcc regression tests.
Is it okay for trunk?
Thanks,
Dehao
gcc/ChangeLog:
2010-11-05 Dehao Chen <dehao@google.com>
* ipa-prop.c (ipa_modify_call_arguments): Set input_location so that
gimplification routines can have right location.
* emit-rtl.c (last_location): Remove unused variable.
[-- Attachment #2: patch.txt --]
[-- Type: text/plain, Size: 1567 bytes --]
Index: gcc/ipa-prop.c
===================================================================
--- gcc/ipa-prop.c (revision 193174)
+++ gcc/ipa-prop.c (working copy)
@@ -2826,7 +2826,9 @@ ipa_modify_call_arguments (struct cgraph_edge *cs,
gimple_stmt_iterator gsi;
tree callee_decl;
int i, len;
+ location_t saved_location = input_location;
+ input_location = gimple_location (stmt);
len = VEC_length (ipa_parm_adjustment_t, adjustments);
vargs = VEC_alloc (tree, heap, len);
callee_decl = !cs ? gimple_call_fndecl (stmt) : cs->callee->symbol.decl;
@@ -3004,6 +3006,7 @@ ipa_modify_call_arguments (struct cgraph_edge *cs,
if (cs)
cgraph_set_call_stmt (cs, new_stmt);
update_ssa (TODO_update_ssa);
+ input_location = saved_location;
free_dominance_info (CDI_DOMINATORS);
}
Index: gcc/emit-rtl.c
===================================================================
--- gcc/emit-rtl.c (revision 193174)
+++ gcc/emit-rtl.c (working copy)
@@ -5937,7 +5937,7 @@ location_t epilogue_location;
/* Hold current location information and last location information, so the
datastructures are built lazily only when some instructions in given
place are needed. */
-static location_t curr_location, last_location;
+static location_t curr_location;
/* Allocate insn location datastructure. */
void
@@ -5945,7 +5945,6 @@ insn_locations_init (void)
{
prologue_location = epilogue_location = 0;
curr_location = UNKNOWN_LOCATION;
- last_location = UNKNOWN_LOCATION;
}
/* At the end of emit stage, clear current location. */
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Cleanup last_location and update input_location in ipa_prop
2012-11-06 1:20 [PATCH] Cleanup last_location and update input_location in ipa_prop Dehao Chen
@ 2012-11-08 17:39 ` Dehao Chen
2012-11-26 15:29 ` Richard Biener
0 siblings, 1 reply; 10+ messages in thread
From: Dehao Chen @ 2012-11-08 17:39 UTC (permalink / raw)
To: GCC Patches
ping...
On Mon, Nov 5, 2012 at 5:20 PM, Dehao Chen <dehao@google.com> wrote:
>
> Hi,
>
> This is a patch to do some obvious cleanup and setting correct
> input_location in ipa_prop (because it invokes gimplification
> routines).
>
> Bootstrapped and passed gcc regression tests.
>
> Is it okay for trunk?
>
> Thanks,
> Dehao
>
> gcc/ChangeLog:
> 2010-11-05 Dehao Chen <dehao@google.com>
>
> * ipa-prop.c (ipa_modify_call_arguments): Set input_location so that
> gimplification routines can have right location.
> * emit-rtl.c (last_location): Remove unused variable.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Cleanup last_location and update input_location in ipa_prop
2012-11-08 17:39 ` Dehao Chen
@ 2012-11-26 15:29 ` Richard Biener
2012-11-26 15:54 ` Dehao Chen
0 siblings, 1 reply; 10+ messages in thread
From: Richard Biener @ 2012-11-26 15:29 UTC (permalink / raw)
To: Dehao Chen; +Cc: GCC Patches
On Thu, Nov 8, 2012 at 6:39 PM, Dehao Chen <dehao@google.com> wrote:
> ping...
The emit-rtl.c hunk is ok. I'm questioning the ipa-prop.c hunk though - what
looks at input_location (nothing outside of the frontend should, really).
Thanks,
Richard.
>
> On Mon, Nov 5, 2012 at 5:20 PM, Dehao Chen <dehao@google.com> wrote:
>>
>> Hi,
>>
>> This is a patch to do some obvious cleanup and setting correct
>> input_location in ipa_prop (because it invokes gimplification
>> routines).
>>
>> Bootstrapped and passed gcc regression tests.
>>
>> Is it okay for trunk?
>>
>> Thanks,
>> Dehao
>>
>> gcc/ChangeLog:
>> 2010-11-05 Dehao Chen <dehao@google.com>
>>
>> * ipa-prop.c (ipa_modify_call_arguments): Set input_location so that
>> gimplification routines can have right location.
>> * emit-rtl.c (last_location): Remove unused variable.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Cleanup last_location and update input_location in ipa_prop
2012-11-26 15:29 ` Richard Biener
@ 2012-11-26 15:54 ` Dehao Chen
2012-11-26 16:10 ` Richard Biener
0 siblings, 1 reply; 10+ messages in thread
From: Dehao Chen @ 2012-11-26 15:54 UTC (permalink / raw)
To: Richard Biener; +Cc: GCC Patches
On Mon, Nov 26, 2012 at 7:28 AM, Richard Biener
<richard.guenther@gmail.com> wrote:
> On Thu, Nov 8, 2012 at 6:39 PM, Dehao Chen <dehao@google.com> wrote:
>> ping...
>
> The emit-rtl.c hunk is ok. I'm questioning the ipa-prop.c hunk though - what
> looks at input_location (nothing outside of the frontend should, really).
ipa_modify_call_arguments invokes force_gimple_operand_gsi, which uses
frontend routines to gimplify expr and uses input_location.
Thanks,
Dehao
>
> Thanks,
> Richard.
>
>>
>> On Mon, Nov 5, 2012 at 5:20 PM, Dehao Chen <dehao@google.com> wrote:
>>>
>>> Hi,
>>>
>>> This is a patch to do some obvious cleanup and setting correct
>>> input_location in ipa_prop (because it invokes gimplification
>>> routines).
>>>
>>> Bootstrapped and passed gcc regression tests.
>>>
>>> Is it okay for trunk?
>>>
>>> Thanks,
>>> Dehao
>>>
>>> gcc/ChangeLog:
>>> 2010-11-05 Dehao Chen <dehao@google.com>
>>>
>>> * ipa-prop.c (ipa_modify_call_arguments): Set input_location so that
>>> gimplification routines can have right location.
>>> * emit-rtl.c (last_location): Remove unused variable.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Cleanup last_location and update input_location in ipa_prop
2012-11-26 15:54 ` Dehao Chen
@ 2012-11-26 16:10 ` Richard Biener
2012-11-26 17:27 ` Dehao Chen
0 siblings, 1 reply; 10+ messages in thread
From: Richard Biener @ 2012-11-26 16:10 UTC (permalink / raw)
To: Dehao Chen; +Cc: GCC Patches
On Mon, Nov 26, 2012 at 4:54 PM, Dehao Chen <dehao@google.com> wrote:
> On Mon, Nov 26, 2012 at 7:28 AM, Richard Biener
> <richard.guenther@gmail.com> wrote:
>> On Thu, Nov 8, 2012 at 6:39 PM, Dehao Chen <dehao@google.com> wrote:
>>> ping...
>>
>> The emit-rtl.c hunk is ok. I'm questioning the ipa-prop.c hunk though - what
>> looks at input_location (nothing outside of the frontend should, really).
>
> ipa_modify_call_arguments invokes force_gimple_operand_gsi, which uses
> frontend routines to gimplify expr and uses input_location.
Can you be more specific? That's the place that needs fixing - there are a lot
more force_gimple_operand callers.
Richard.
> Thanks,
> Dehao
>
>>
>> Thanks,
>> Richard.
>>
>>>
>>> On Mon, Nov 5, 2012 at 5:20 PM, Dehao Chen <dehao@google.com> wrote:
>>>>
>>>> Hi,
>>>>
>>>> This is a patch to do some obvious cleanup and setting correct
>>>> input_location in ipa_prop (because it invokes gimplification
>>>> routines).
>>>>
>>>> Bootstrapped and passed gcc regression tests.
>>>>
>>>> Is it okay for trunk?
>>>>
>>>> Thanks,
>>>> Dehao
>>>>
>>>> gcc/ChangeLog:
>>>> 2010-11-05 Dehao Chen <dehao@google.com>
>>>>
>>>> * ipa-prop.c (ipa_modify_call_arguments): Set input_location so that
>>>> gimplification routines can have right location.
>>>> * emit-rtl.c (last_location): Remove unused variable.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Cleanup last_location and update input_location in ipa_prop
2012-11-26 16:10 ` Richard Biener
@ 2012-11-26 17:27 ` Dehao Chen
2012-11-26 22:55 ` Dehao Chen
0 siblings, 1 reply; 10+ messages in thread
From: Dehao Chen @ 2012-11-26 17:27 UTC (permalink / raw)
To: Richard Biener; +Cc: GCC Patches
On Mon, Nov 26, 2012 at 8:10 AM, Richard Biener
<richard.guenther@gmail.com> wrote:
> On Mon, Nov 26, 2012 at 4:54 PM, Dehao Chen <dehao@google.com> wrote:
>> On Mon, Nov 26, 2012 at 7:28 AM, Richard Biener
>> <richard.guenther@gmail.com> wrote:
>>> On Thu, Nov 8, 2012 at 6:39 PM, Dehao Chen <dehao@google.com> wrote:
>>>> ping...
>>>
>>> The emit-rtl.c hunk is ok. I'm questioning the ipa-prop.c hunk though - what
>>> looks at input_location (nothing outside of the frontend should, really).
>>
>> ipa_modify_call_arguments invokes force_gimple_operand_gsi, which uses
>> frontend routines to gimplify expr and uses input_location.
>
> Can you be more specific? That's the place that needs fixing - there are a lot
> more force_gimple_operand callers.
You are right, the patch is updated as follows:
Index: gcc/ipa-prop.c
===================================================================
--- gcc/ipa-prop.c (revision 193203)
+++ gcc/ipa-prop.c (working copy)
@@ -2870,7 +2870,8 @@ ipa_modify_call_arguments (struct cgraph_edge *cs,
gcc_checking_assert (adj->offset % BITS_PER_UNIT == 0);
base = gimple_call_arg (stmt, adj->base_index);
- loc = EXPR_LOCATION (base);
+ loc = DECL_P (base) ? DECL_SOURCE_LOCATION (base) :
+ EXPR_LOCATION (base);
if (TREE_CODE (base) != ADDR_EXPR
&& POINTER_TYPE_P (TREE_TYPE (base)))
Thanks,
Dehao
>
> Richard.
>
>> Thanks,
>> Dehao
>>
>>>
>>> Thanks,
>>> Richard.
>>>
>>>>
>>>> On Mon, Nov 5, 2012 at 5:20 PM, Dehao Chen <dehao@google.com> wrote:
>>>>>
>>>>> Hi,
>>>>>
>>>>> This is a patch to do some obvious cleanup and setting correct
>>>>> input_location in ipa_prop (because it invokes gimplification
>>>>> routines).
>>>>>
>>>>> Bootstrapped and passed gcc regression tests.
>>>>>
>>>>> Is it okay for trunk?
>>>>>
>>>>> Thanks,
>>>>> Dehao
>>>>>
>>>>> gcc/ChangeLog:
>>>>> 2010-11-05 Dehao Chen <dehao@google.com>
>>>>>
>>>>> * ipa-prop.c (ipa_modify_call_arguments): Set input_location so that
>>>>> gimplification routines can have right location.
>>>>> * emit-rtl.c (last_location): Remove unused variable.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Cleanup last_location and update input_location in ipa_prop
2012-11-26 17:27 ` Dehao Chen
@ 2012-11-26 22:55 ` Dehao Chen
2012-11-27 9:47 ` Richard Biener
0 siblings, 1 reply; 10+ messages in thread
From: Dehao Chen @ 2012-11-26 22:55 UTC (permalink / raw)
To: Richard Biener; +Cc: GCC Patches
The new patch is attached. Bootstrapped and passed gcc regression test.
Ok for trunk?
Thanks,
Dehao
gcc/ChangeLog:
2010-11-05 Dehao Chen <dehao@google.com>
* ipa-prop.c (ipa_modify_call_arguments): Set loc correctly.
* emit-rtl.c (last_location): Remove unused variable.
Index: gcc/emit-rtl.c
===================================================================
--- gcc/emit-rtl.c (revision 193203)
+++ gcc/emit-rtl.c (working copy)
@@ -5937,7 +5937,7 @@ location_t epilogue_location;
/* Hold current location information and last location information, so the
datastructures are built lazily only when some instructions in given
place are needed. */
-static location_t curr_location, last_location;
+static location_t curr_location;
/* Allocate insn location datastructure. */
void
@@ -5945,7 +5945,6 @@ insn_locations_init (void)
{
prologue_location = epilogue_location = 0;
curr_location = UNKNOWN_LOCATION;
- last_location = UNKNOWN_LOCATION;
}
/* At the end of emit stage, clear current location. */
Index: gcc/ipa-prop.c
===================================================================
--- gcc/ipa-prop.c (revision 193203)
+++ gcc/ipa-prop.c (working copy)
@@ -2870,7 +2870,8 @@ ipa_modify_call_arguments (struct cgraph_edge *cs,
gcc_checking_assert (adj->offset % BITS_PER_UNIT == 0);
base = gimple_call_arg (stmt, adj->base_index);
- loc = EXPR_LOCATION (base);
+ loc = DECL_P (base) ? DECL_SOURCE_LOCATION (base)
+ : EXPR_LOCATION (base);
if (TREE_CODE (base) != ADDR_EXPR
&& POINTER_TYPE_P (TREE_TYPE (base)))
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Cleanup last_location and update input_location in ipa_prop
2012-11-26 22:55 ` Dehao Chen
@ 2012-11-27 9:47 ` Richard Biener
2012-11-27 14:54 ` Dehao Chen
0 siblings, 1 reply; 10+ messages in thread
From: Richard Biener @ 2012-11-27 9:47 UTC (permalink / raw)
To: Dehao Chen; +Cc: GCC Patches
On Mon, Nov 26, 2012 at 11:54 PM, Dehao Chen <dehao@google.com> wrote:
> The new patch is attached. Bootstrapped and passed gcc regression test.
>
> Ok for trunk?
Hmm ... this merely avoids some UNKNOWN_LOCATIONs which should
be ok. I think the issue is that gimplify_expr does:
saved_location = input_location;
if (save_expr != error_mark_node
&& EXPR_HAS_LOCATION (*expr_p))
input_location = EXPR_LOCATION (*expr_p);
thus it retains input_location from previous recursive invocations (that's ok).
But it doesn't seem to be the case that input_location is UNKNOWN_LOCATION
during GIMPLE passes (which it really should be ...). So to fix the
gimplification
issue I think that gimplify_ctx should have a location (initialized to
UNKNOWN_LOCATION), which it saves/restores across its push/pop
operation.
Or of course that inside the pass manager before executing passes assert
that input_location is UNKNOWN_LOCATION and fix up things according
to that. First offender is in cgraphunit.c in cgraph_analyze_function:
location_t saved_loc = input_location;
input_location = DECL_SOURCE_LOCATION (decl);
that should be totally unnecessary (input_location shoud be and stay
UNKNOWN_LOCATION). And finalize_compilation_unit (the last thing
the frontends should call) should have input_location = UNKNOWN_LOCATION
right at the point it clears current_function_decl / cfun.
I'd prefer the 2nd approach (maybe without the assert as we are in stage3
already, but the assert would certainly help). And places in the middle-end
that set input_location for purpose of (re-)gimplifying should use the location
in the gimplify ctx (which would need to be added) instead of setting
input_location.
Maybe as a first try set input_location to UNKNOWN_LOCATION in
finalize_compilation_unit.
Thanks,
Richard.
> Thanks,
> Dehao
>
> gcc/ChangeLog:
> 2010-11-05 Dehao Chen <dehao@google.com>
>
> * ipa-prop.c (ipa_modify_call_arguments): Set loc correctly.
> * emit-rtl.c (last_location): Remove unused variable.
>
> Index: gcc/emit-rtl.c
> ===================================================================
> --- gcc/emit-rtl.c (revision 193203)
> +++ gcc/emit-rtl.c (working copy)
> @@ -5937,7 +5937,7 @@ location_t epilogue_location;
> /* Hold current location information and last location information, so the
> datastructures are built lazily only when some instructions in given
> place are needed. */
> -static location_t curr_location, last_location;
> +static location_t curr_location;
>
> /* Allocate insn location datastructure. */
> void
> @@ -5945,7 +5945,6 @@ insn_locations_init (void)
> {
> prologue_location = epilogue_location = 0;
> curr_location = UNKNOWN_LOCATION;
> - last_location = UNKNOWN_LOCATION;
> }
>
> /* At the end of emit stage, clear current location. */
> Index: gcc/ipa-prop.c
> ===================================================================
> --- gcc/ipa-prop.c (revision 193203)
> +++ gcc/ipa-prop.c (working copy)
> @@ -2870,7 +2870,8 @@ ipa_modify_call_arguments (struct cgraph_edge *cs,
>
> gcc_checking_assert (adj->offset % BITS_PER_UNIT == 0);
> base = gimple_call_arg (stmt, adj->base_index);
> - loc = EXPR_LOCATION (base);
> + loc = DECL_P (base) ? DECL_SOURCE_LOCATION (base)
> + : EXPR_LOCATION (base);
>
> if (TREE_CODE (base) != ADDR_EXPR
> && POINTER_TYPE_P (TREE_TYPE (base)))
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Cleanup last_location and update input_location in ipa_prop
2012-11-27 9:47 ` Richard Biener
@ 2012-11-27 14:54 ` Dehao Chen
2012-11-27 15:22 ` Richard Biener
0 siblings, 1 reply; 10+ messages in thread
From: Dehao Chen @ 2012-11-27 14:54 UTC (permalink / raw)
To: Richard Biener; +Cc: GCC Patches
On Tue, Nov 27, 2012 at 1:46 AM, Richard Biener
<richard.guenther@gmail.com> wrote:
> On Mon, Nov 26, 2012 at 11:54 PM, Dehao Chen <dehao@google.com> wrote:
>> The new patch is attached. Bootstrapped and passed gcc regression test.
>>
>> Ok for trunk?
>
> Hmm ... this merely avoids some UNKNOWN_LOCATIONs which should
> be ok. I think the issue is that gimplify_expr does:
>
> saved_location = input_location;
> if (save_expr != error_mark_node
> && EXPR_HAS_LOCATION (*expr_p))
> input_location = EXPR_LOCATION (*expr_p);
>
> thus it retains input_location from previous recursive invocations (that's ok).
>
> But it doesn't seem to be the case that input_location is UNKNOWN_LOCATION
> during GIMPLE passes (which it really should be ...). So to fix the
> gimplification
> issue I think that gimplify_ctx should have a location (initialized to
> UNKNOWN_LOCATION), which it saves/restores across its push/pop
> operation.
>
> Or of course that inside the pass manager before executing passes assert
> that input_location is UNKNOWN_LOCATION and fix up things according
> to that. First offender is in cgraphunit.c in cgraph_analyze_function:
>
> location_t saved_loc = input_location;
> input_location = DECL_SOURCE_LOCATION (decl);
>
> that should be totally unnecessary (input_location shoud be and stay
> UNKNOWN_LOCATION). And finalize_compilation_unit (the last thing
> the frontends should call) should have input_location = UNKNOWN_LOCATION
> right at the point it clears current_function_decl / cfun.
>
> I'd prefer the 2nd approach (maybe without the assert as we are in stage3
> already, but the assert would certainly help). And places in the middle-end
> that set input_location for purpose of (re-)gimplifying should use the location
> in the gimplify ctx (which would need to be added) instead of setting
> input_location.
>
> Maybe as a first try set input_location to UNKNOWN_LOCATION in
> finalize_compilation_unit.
>
> Thanks,
> Richard.
Shall we check in this patch first, and have another patch to reset
input_location?
Thanks,
Dehao
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Cleanup last_location and update input_location in ipa_prop
2012-11-27 14:54 ` Dehao Chen
@ 2012-11-27 15:22 ` Richard Biener
0 siblings, 0 replies; 10+ messages in thread
From: Richard Biener @ 2012-11-27 15:22 UTC (permalink / raw)
To: Dehao Chen; +Cc: GCC Patches
On Tue, Nov 27, 2012 at 3:54 PM, Dehao Chen <dehao@google.com> wrote:
> On Tue, Nov 27, 2012 at 1:46 AM, Richard Biener
> <richard.guenther@gmail.com> wrote:
>> On Mon, Nov 26, 2012 at 11:54 PM, Dehao Chen <dehao@google.com> wrote:
>>> The new patch is attached. Bootstrapped and passed gcc regression test.
>>>
>>> Ok for trunk?
>>
>> Hmm ... this merely avoids some UNKNOWN_LOCATIONs which should
>> be ok. I think the issue is that gimplify_expr does:
>>
>> saved_location = input_location;
>> if (save_expr != error_mark_node
>> && EXPR_HAS_LOCATION (*expr_p))
>> input_location = EXPR_LOCATION (*expr_p);
>>
>> thus it retains input_location from previous recursive invocations (that's ok).
>>
>> But it doesn't seem to be the case that input_location is UNKNOWN_LOCATION
>> during GIMPLE passes (which it really should be ...). So to fix the
>> gimplification
>> issue I think that gimplify_ctx should have a location (initialized to
>> UNKNOWN_LOCATION), which it saves/restores across its push/pop
>> operation.
>>
>> Or of course that inside the pass manager before executing passes assert
>> that input_location is UNKNOWN_LOCATION and fix up things according
>> to that. First offender is in cgraphunit.c in cgraph_analyze_function:
>>
>> location_t saved_loc = input_location;
>> input_location = DECL_SOURCE_LOCATION (decl);
>>
>> that should be totally unnecessary (input_location shoud be and stay
>> UNKNOWN_LOCATION). And finalize_compilation_unit (the last thing
>> the frontends should call) should have input_location = UNKNOWN_LOCATION
>> right at the point it clears current_function_decl / cfun.
>>
>> I'd prefer the 2nd approach (maybe without the assert as we are in stage3
>> already, but the assert would certainly help). And places in the middle-end
>> that set input_location for purpose of (re-)gimplifying should use the location
>> in the gimplify ctx (which would need to be added) instead of setting
>> input_location.
>>
>> Maybe as a first try set input_location to UNKNOWN_LOCATION in
>> finalize_compilation_unit.
>>
>> Thanks,
>> Richard.
>
> Shall we check in this patch first, and have another patch to reset
> input_location?
Sure, that works for me. It might be harder for you to verify that
doing that also fixes the issue though ;)
Richard.
> Thanks,
> Dehao
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2012-11-27 15:22 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-11-06 1:20 [PATCH] Cleanup last_location and update input_location in ipa_prop Dehao Chen
2012-11-08 17:39 ` Dehao Chen
2012-11-26 15:29 ` Richard Biener
2012-11-26 15:54 ` Dehao Chen
2012-11-26 16:10 ` Richard Biener
2012-11-26 17:27 ` Dehao Chen
2012-11-26 22:55 ` Dehao Chen
2012-11-27 9:47 ` Richard Biener
2012-11-27 14:54 ` Dehao Chen
2012-11-27 15:22 ` Richard Biener
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).