On Thu, 2013-11-14 at 00:37 -0700, Jeff Law wrote:
On 10/31/13 10:26, David Malcolm wrote:
* gimple.h (is_a_helper <const gimple_statement_with_ops>::test): New.
(is_a_helper <gimple_statement_with_ops>::test): New.
(is_a_helper <const gimple_statement_with_memory_ops>::test): New.
(is_a_helper <gimple_statement_with_memory_ops>::test): New.
OK with the usual conditions. Check with Andrew as to the location of
these helpers since he's in the middle of ripping apart gimple.h.
The helpers become used in accessors such as gimple_use_ops,
gimple_vuse, gimple_vdef etc as of the automated patch (patch 3 in the
series [1]), so presumably the helpers need to be in the same file as
those accessors. The accessors are currently in gimple.h, and hence I
also put the helpers within gimple.h (earlier within the header).
Andrew, do you plan to move the accessor functions out of gimple.h? If
so, where should the is_a_helper fns land?
Dave
[1] hence these patches need to be committed at the same time, whether
than means a single "revision" or not; I split them up conceptually for
ease of review.