[PATCH] D97070: [ValueTypes] Assert if changeVectorElementType is called on a simple type with an extended element type.

Pengfei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 19 17:03:36 PST 2021


pengfei accepted this revision.
pengfei added a comment.
This revision is now accepted and ready to land.

LGTM.



================
Comment at: llvm/lib/CodeGen/ValueTypes.cpp:18
 EVT EVT::changeExtendedTypeToInteger() const {
+  assert(isExtended() && "Type is not extended!");
   LLVMContext &Context = LLVMTy->getContext();
----------------
It seems all the 3 functions are checked for not simple before called. But adding assertions are fine.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97070



More information about the llvm-commits mailing list