[llvm] [IR] Add `llvm.sincos` intrinsic (PR #109825)

Benjamin Maxwell via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 28 03:04:32 PDT 2024


================
@@ -4448,6 +4470,8 @@ void DAGTypeLegalizer::WidenVectorResult(SDNode *N, unsigned ResNo) {
     if (!TLI.isOperationLegalOrCustomOrPromote(N->getOpcode(), WideVecVT) &&
         TLI.isOperationExpand(N->getOpcode(), VT.getScalarType())) {
       Res = DAG.UnrollVectorOp(N, WideVecVT.getVectorNumElements());
+      if (N->getNumValues() == 2)
----------------
MacDue wrote:

I'm not sure this should be implemented now. `DAG.UnrollVectorOp()` also asserts that the number of results is 1 or 2. I could just add an assert after the `if` that the number of results is 1 and leave further extension to when someone needs it? 

https://github.com/llvm/llvm-project/pull/109825


More information about the llvm-commits mailing list