aemerson wrote: > I think this sounds OK. LGTM > > (Which of bitcast or load/store is considered the most fundamental for > v4i1/v8i1? I think I would have expected in GISel the loads to be converted > to a i4/i8 load with bitcast, and the bitcast legalizes however it does. It > could obviously go the other way where a bitcast is just legalized to > load+store. I wasn't sure why the v4i1 load needed to produce an extending > load just just to scalarize again, but perhaps it is necessary to get past > legalization successfully, I haven't looked a lot into it lately. )
I think for loads of v4i1 we should do as you say and bitcast to i4 and then legalize the bitcast. It looks like we currently don't do that and instead we try to lower it, which ends up failing. https://github.com/llvm/llvm-project/pull/121185 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
