> Ugh. I don't care that much :-) If you want to handle it, go ahead. I > would suggest a comment indicating why we've chosen to handle it.
OK, I can do that. > The only question left is would it be better to handle it in > extract_bit_field to catch other cases, or is that too late? I put it there because there is a similar bitsize == 0 case just above: if (ext_mode == BLKmode) { if (target == 0) target = assign_temp (type, 1, 1); if (bitsize == 0) return target; Not clear how we can reach here though, I could add a comment if I find. :-) -- Eric Botcazou