Re: [PATCH 012/236] Convert DF_REF_INSN to a function for now

2014-08-19 Thread David Malcolm
On Tue, 2014-08-12 at 15:20 -0600, Jeff Law wrote: On 08/06/14 11:19, David Malcolm wrote: DF_REF_INSN looks up the insn field of the referenced df_insn_info. This will eventually be an rtx_insn *, but for now is just an rtx. As further scaffolding: for now, convert DF_REF_INSN to a

Re: [PATCH 012/236] Convert DF_REF_INSN to a function for now

2014-08-14 Thread David Malcolm
On Wed, 2014-08-13 at 20:55 -0600, Jeff Law wrote: On 08/13/14 18:11, David Malcolm wrote: On Wed, 2014-08-13 at 14:34 -0600, Jeff Law wrote: On 08/13/14 14:28, David Malcolm wrote: Thanks. Although this function gets converted back to a macro in patch 191, I just realized that in the

Re: [PATCH 012/236] Convert DF_REF_INSN to a function for now

2014-08-13 Thread David Malcolm
On Tue, 2014-08-12 at 15:20 -0600, Jeff Law wrote: On 08/06/14 11:19, David Malcolm wrote: DF_REF_INSN looks up the insn field of the referenced df_insn_info. This will eventually be an rtx_insn *, but for now is just an rtx. As further scaffolding: for now, convert DF_REF_INSN to a

Re: [PATCH 012/236] Convert DF_REF_INSN to a function for now

2014-08-13 Thread Jeff Law
On 08/13/14 14:28, David Malcolm wrote: Thanks. Although this function gets converted back to a macro in patch 191, I just realized that in the meantime that it's not inlined, as is the case for some of the other macro-function conversions in patches 13-16. Do I need to convert them to inline

Re: [PATCH 012/236] Convert DF_REF_INSN to a function for now

2014-08-13 Thread David Malcolm
On Wed, 2014-08-13 at 14:34 -0600, Jeff Law wrote: On 08/13/14 14:28, David Malcolm wrote: Thanks. Although this function gets converted back to a macro in patch 191, I just realized that in the meantime that it's not inlined, as is the case for some of the other macro-function conversions

Re: [PATCH 012/236] Convert DF_REF_INSN to a function for now

2014-08-13 Thread Jeff Law
On 08/13/14 18:11, David Malcolm wrote: On Wed, 2014-08-13 at 14:34 -0600, Jeff Law wrote: On 08/13/14 14:28, David Malcolm wrote: Thanks. Although this function gets converted back to a macro in patch 191, I just realized that in the meantime that it's not inlined, as is the case for some of

Re: [PATCH 012/236] Convert DF_REF_INSN to a function for now

2014-08-12 Thread Jeff Law
On 08/06/14 11:19, David Malcolm wrote: DF_REF_INSN looks up the insn field of the referenced df_insn_info. This will eventually be an rtx_insn *, but for now is just an rtx. As further scaffolding: for now, convert DF_REF_INSN to a function, adding a checked downcast to rtx_insn *. This can

[PATCH 012/236] Convert DF_REF_INSN to a function for now

2014-08-06 Thread David Malcolm
DF_REF_INSN looks up the insn field of the referenced df_insn_info. This will eventually be an rtx_insn *, but for now is just an rtx. As further scaffolding: for now, convert DF_REF_INSN to a function, adding a checked downcast to rtx_insn *. This can eventually be converted back to macro when