[PATCH] D80466: [X86] Improve i8 + 'slow' i16 funnel shift codegen

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 23 08:27:57 PDT 2020


lebedev.ri added a comment.

LGTM

For `fshl` case, we could introduce some more ILP: http://volta.cs.utah.edu:8080/z/UJ6viM
https://godbolt.org/z/xsJgPb  https://godbolt.org/z/5W26NV
Not sure it would be an improvement?
As a sidenote, we clearly don't fold to either variant in DAGCombiner.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80466/new/

https://reviews.llvm.org/D80466





More information about the llvm-commits mailing list