Thanks for the review! Eric Botcazou <ebotcazou@libertysurf.fr> writes: >> This is OK for mainline, modulo the location of extract_low_bits: why not >> put it in emit-rtl.c like gen_lowpart_*? > > Maybe expmed.c as a middle ground? Sure, expmed.c's fine with me. I agree it makes more sense than optabs. I'll retest with that change. Richard