> They don't need to be kept synchronised as such. It's fine for the index > to allow more than must_be_index_p. But if you're not keen on the current > structure, does the following look better? Tested on x86_64-linux-gnu. > > Thanks, > Richard > > > gcc/ > * rtlanal.c (must_be_base_p, must_be_index_p): Delete. > (binary_scale_code_p, get_base_term, get_index_term): New functions. > (set_address_segment, set_address_base, set_address_index) > (set_address_disp): Accept the argument unconditionally. > (baseness): Remove must_be_base_p and must_be_index_p checks. > (decompose_normal_address): Classify as much as possible in the > main loop.
Yes, fine by me, thanks. -- Eric Botcazou
