On Fri, Sep 2, 2011 at 3:45 PM, Ilya Enkovich <enkovich....@gmail.com> wrote: > 2011/9/2 Richard Guenther <richard.guent...@gmail.com>: >>On Fri, Sep 2, 2011 at 2:52 PM, Uros Bizjak <ubiz...@gmail.com> wrote: >>> >>> I assume that you need to split tune attribute to int and FP part to >>> handle reassociation for other targets, since Atom handles both in the >>> same way. >>> >>> Please also describe function return value in the comment (and perhaps >>> in documentation, too). >>> >>> OK with this addition. >> >> Btw, I would expect integer add and integer multiply to have different >> settings for some targets which would mean splitting this up even >> further ... > > Which tune attributes are meant here? Is it X86_TUNE_REASSOC_* flags > or new command line param?
The X86_TUNE_REASSOC_* flags. The setting surely depends on the number of available execution units and/or whether the instructions are pipelined or not. Richard.