On 09/20/2011 01:46 AM, Bingfeng Mei wrote: > if (MEM_P (rtlx) && MEM_P (rtly) > && MEM_ADDR_SPACE (rtlx) != MEM_ADDR_SPACE (rtly)) > - return 0; > + { > + if (!targetm.addr_space.subset_p (MEM_ADDR_SPACE (rtlx), > + MEM_ADDR_SPACE (rtly)) > + && !targetm.addr_space.subset_p (MEM_ADDR_SPACE (rtly), > + MEM_ADDR_SPACE (rtlx))) > + return 1; > + else > + return 0; > + }
You have 4 copies of this. Please pull it out into a new predicate function. r~