On 11/17/2017 08:04 AM, Pedro Alves wrote: > On 11/17/2017 04:49 AM, Jeff Law wrote: > >> + /* We do not allow copying this object or initializing one from another. >> */ >> + evrp_dom_walker (const evrp_dom_walker &); >> + evrp_dom_walker& operator= (const evrp_dom_walker &); >> + > > Note you can use include/ansidecl.h's DISABLE_COPY_AND_ASSIGN for > this [1], and then you don't need the comment, as it's > self-documenting. > > [1] - https://marc.info/?l=gcc-patches&m=150549025810729&w=2 > (gcc.gnu.org seems to be unreachable for me today.) Thanks for the pointer. I'll update the patch to use the macro.
Jeff