fhahn added inline comments.
================
Comment at: clang/lib/CodeGen/CGExprScalar.cpp:1324
+
+ if (SrcElementTy->isFloatTy() || SrcElementTy->isDoubleTy()) {
+ QualType DstElementType =
DstType->castAs<MatrixType>()->getElementType();
----------------
SaurabhJha wrote:
> fhahn wrote:
> > I think we should support all floating point type here
> > (`isFloatingPointTy`). Basically we want the same rules as for integer
> > types (around line 1418). Perhaps it would be possible to generalize this
> > code in a separate function that takes the LLVM types & the element
> > type/scalar types?
> I think we should move this code to `MatrixBuilder` but then generalising
> floating point after that would be problematic since the code is splitted
> across this function and in MatrixBuilder, right?
That's a good point. Sharing the logic with the scalar case in Clang directly
seems much more valuable than having it in the MatrixBuilder IMO.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99037/new/
https://reviews.llvm.org/D99037
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
