ilya-biryukov wrote:

@kadircet let's team up tomorrow and poke at what's happening with the smaller 
reproducer to figure out if this is the right fix?

@AaronBallman I was initially concerned that it would be incorrect to consider 
the compound literal expression a full expression even if it fixes the crash. 
After re-reading the code now, I'm not sure how to properly test that as C++ 
does not have those (right?) and so I'd probably spend some time poking at GCC 
and Clang's codegen to see if they exhibit the signs of full expressions.

If you feel this fix is correct (or at least won't hurt too much), we can 
definitely land this ASAP. Help with reviewing this or steering it in the right 
direction is greatly appreciated.

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

Reply via email to