steffenlarsen wrote: > > > Do we need similar checks anywhere else we check isPackedVectorBoolType? > > > > > > Luckily that case was already handled > > [here](https://github.com/llvm/llvm-project/pull/161796/files#diff-328fcdcf882c121d5c56fdc6cd56c43fdbbb00da171da70f3ba6982c43686a63L2097-L2098) > > and now > > [here](https://github.com/llvm/llvm-project/pull/161796/files#diff-328fcdcf882c121d5c56fdc6cd56c43fdbbb00da171da70f3ba6982c43686a63R2098-R2099). > > 😄 > > Sorry, I meant, code that isn't currently touched by your patch. > ExprConstant, CGDebugInfo, etc. Not that you need to fix everything in one > patch, but I'd like some idea of the scope of the full set of patches.
Ah, my bad! Turns out my Monday morning reading comprehension is on part with a 3rd grader's. I will have a look through the places that have `isPackedVectorBoolType` and see if we more handling. https://github.com/llvm/llvm-project/pull/161796 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
