[PATCH] D48491: [X86] Select BEXTR when there is only BMI1.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 24 23:21:06 PDT 2018


craig.topper added a comment.

Matching multiple nodes in isel patterns without one use checks and then emitting multiple instructions for the pattern seems like dangerous territory. Is there more canonicalizing we should be doing in DAG combines to get these sequences into a form we can handle in isel?


Repository:
  rL LLVM

https://reviews.llvm.org/D48491





More information about the llvm-commits mailing list