mizvekov added a comment. In D103720#2806147 <https://reviews.llvm.org/D103720#2806147>, @aaron.ballman wrote:
> (esp if someone in the future wants to add `Expr::isRValue()` with the C++ > meaning of prvalue || xvalue). @rsmith actually suggested we add `isRValue` as a second step here, and I think it's a good idea, but I was also worried a bit about the practical issue: Others rebasing patches on top of that change without realizing it, and introducing possibly subtle issues :) We could wait just a little bit for that and let others just soak in the build breakage meanwhile. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D103720/new/ https://reviews.llvm.org/D103720 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
