tejohnson added a comment. lgtm with the VecLibDesc references removed.
================ Comment at: llvm/lib/Analysis/TargetLibraryInfo.cpp:596 memcpy(AvailableArray, TLI.AvailableArray, sizeof(AvailableArray)); return *this; } ---------------- wenlei wrote: > tejohnson wrote: > > This is missing copying of the VecLibDescs (looks like I missed this as > > well originally). > Now I remembered why this was missed from my side, before my patch, > `VecLibDescs` isn't copied here either, which seems like a bug? Same for the > move one below. I assume you mean the VectorDescs/ScalarDescs vectors. You are right, even before your D77632 patch they were not being copied here or below. Looks like there is an existing bug there that no one noticed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77952/new/ https://reviews.llvm.org/D77952 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
