tbaederr wrote: > The assertion I think is a good addition too, but I feel like it should go in > a separate patch, especially since apparently that’s causing issues elsewhere > (possibly due to more missing lvalue-to-rvalue conversions).
I'm not sure what to do about the assertion in the string_view case above, because `DefaultLvalueConversion` doesn't do anything in those cases, i.e. where the lvalue is of struct type. https://github.com/llvm/llvm-project/pull/227998 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
