[PATCH] D28194: [ARM] Classification Improvements to ARM Sched-Models. NFCI.
    Renato Golin via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Jan  6 06:10:03 PST 2017
    
    
  
rengolin added a comment.
In https://reviews.llvm.org/D28194#637818, @javed.absar wrote:
> I tried not to club  too many of the sched-defs into one bucket because then sub-targets wouldn't be able to specify different latencies/resources if there was indeed a difference in the pipeline behavior for different instruction classes (e.g. between MUL32 and MUL64).
I see where you're coming from, but I'd rather have that differentiation done by the sub-architectures when they need, than just explode the combinations now, and having to apply larger patches every time one sub-arch needs one cost changed. :)
--renato
https://reviews.llvm.org/D28194
    
    
More information about the llvm-commits
mailing list