[PATCH] D129537: [X86][DAGISel] Don't widen shuffle element with AVX512

LuoYuanke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 25 20:20:03 PDT 2022


LuoYuanke added a comment.

In D129537#3676257 <https://reviews.llvm.org/D129537#3676257>, @RKSimon wrote:

> Yeah - that was what I saw as well - if you want to get this in for 15.x I'd recommend going back to the old switch statement - then investigate the binop general case later (if you solve it soon enough you request a merge)

Thank Simon for the review. All the comments are valuable. Let me land the old switch statement version first and then investigate the general binop.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129537



More information about the llvm-commits mailing list