[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