[PATCH] D79399: [CodeGen] Fix incorrect uses of getVectorNumElements()

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 5 02:06:40 PDT 2020


david-arm created this revision.
david-arm added reviewers: kmclaughlin, sdesmalen, ctetreau.
Herald added subscribers: llvm-commits, hiraditya, kristof.beyls.
Herald added a reviewer: rengolin.
Herald added a project: LLVM.

I have fixed up some places in SelectionDAG::getNode() where we
used to assert that the number of vector elements for two types
are the same. I have changed such cases to assert that the
element counts are the same instead. I've added new tests that
exercise the code paths for all the extends and truncations.

For the ISD::SETCC case I fixed this code path is exercised by
these existing tests:

  CodeGen/AArch64/sve-fcmp.ll
  CodeGen/AArch64/sve-intrinsics-int-compares-with-imm.ll


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D79399

Files:
  llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
  llvm/test/CodeGen/AArch64/sve-ext-trunc.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D79399.262025.patch
Type: text/x-patch
Size: 7807 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200505/9412b0bc/attachment.bin>


More information about the llvm-commits mailing list