https://github.com/AaronBallman commented:

I think the current changes are reasonable to land for Clang 18 so we get the 
fix out to users. I don't have strong opinions on pushing this down into LLVM 
or outlining the function like John suggested; either approach is fine by me. I 
think outlining the function might make sense but I don't think it's a 
requirement either (perhaps add a FIXME comment suggesting it?).

https://github.com/llvm/llvm-project/pull/78330
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to