On 05/08/2015 12:41 PM, Jeff Law wrote: > I'm going to assume the include header order juggling/additions were all > necessary.
Correct -- target.h now uses vec<>, which means that vec.h must be included first. In the olden days I'd have just put the vec.h include at the top of target.h. ;-) > Presumably we don't have a symbolic return value from decode_reg_name. > That'd > be a good cleanup for someone. Correct. Since the beginning of time, it would seem. > Much of the cfgexpand bits were painful to read. I did my best, things look > reasonable, but that's largely from reading the comments and scanning the > relevant code -- trying to match up old behaviour and new behaviour across the > board was tough. I realize this mess isn't of your making :-) Yeah. Sadly there was no smaller step; you can't half-covert a variable from lists to vectors. And since the existing target hook used the lists, they had to be done at the same time. r~