Re: RFA: hookize ADJUST_INSN_LENGTH

2012-11-04 Thread amylaar
Quoting Joern Rennecke : The return value is a pointer to a function, to be used for adjusting instruction length in the place of ADJUST_INSN_LENGTHS - and where it is missing, or zero if no such functionis required. (I first thought of having separate target hooks for everything, but then I re

Re: RFA: Process '*' in '@'-output-template alternatives

2012-09-19 Thread amylaar
Quoting Georg-Johann Lay : + * return stack_mem_p (operands[0]) ? \"push 0" : \"clrmem %0\"; ---^ Isn't there a backslash missing? Attached is the amended patch. Again, bootstrapped on i686-pc-linux-gnu . 2011-09-19 J"orn Rennecke

Re: RFA: Process '*' in '@'-output-template alternatives

2012-09-19 Thread amylaar
Quoting Georg-Johann Lay : + * return stack_mem_p (operands[0]) ? \"push 0" : \"clrmem %0\"; ---^ Isn't there a backslash missing? Oops, yes, there is.

Re: RFA: Fix dse / postreload not to bypass add expanders

2011-11-04 Thread amylaar
Quoting Paolo Bonzini : Unless I'm missing something, this is going all the way down to the end of the function, bypassing the CFG, so it is neither efficient nor correct. new_insn hasn't been emitted yet, hence it is a single insn or a short chain of insn to implement a no-op move (for a post