pshung added a comment.
@khchen
This patch doesn't generate the overloaded APIs for "vreinterpret". Any concern?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100391/new/
https://reviews.llvm.org/D100391
__
craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100391/new/
https://reviews.llvm.org/D100391
___
khchen added inline comments.
Comment at: clang/include/clang/Basic/riscv_vector.td:1304
+// Reinterpret between different SEW under the same LMUL
+foreach dst_sew = ["(FixedLog2SEW:3)", "(FixedLog2SEW:4)",
"(FixedLog2SEW:5)",
+ "(FixedLog2SEW:6)"]
craig.topper added inline comments.
Comment at: clang/include/clang/Basic/riscv_vector.td:97
+// type (SEW and LMUL), and computes another vector type which only
changed
+// SEW as given value. Ingore to define a new builtin if its qeuivalent
+// type has illegal
