[PATCH] D67549: [IntrinsicEmitter] Add overloaded types for SVE intrinsics (Subdivide2 & Subdivide4)

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 19 00:38:32 PDT 2019


sdesmalen added inline comments.


================
Comment at: include/llvm/IR/DerivedTypes.h:500
+    default:
+      assert(0 && "Cannot create narrower fp vector element type");
+      break;
----------------
nit: Use `llvm_unreachable("message")` instead of `assert(0 && "message")`


Repository:
  rL LLVM

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

https://reviews.llvm.org/D67549





More information about the llvm-commits mailing list