Actually kludging it by just removing the assert isn't going to work. I'll ping
Pengfei to see about developing a patch for this problem.
> -Original Message-
> From: dmajor via Phabricator
> Sent: Tuesday, October 27, 2020 11:58 AM
> To: Blower, Melanie I ; sepavl...@gmail.com;
> rjmcc
Thank you. I don't know the solution but I can remove the assert and create a
bug report to have it resolved.
> -Original Message-
> From: dmajor via Phabricator
> Sent: Tuesday, October 27, 2020 11:58 AM
> To: Blower, Melanie I ; sepavl...@gmail.com;
> rjmcc...@gmail.com; rich...@metaf
Sorry I don't understand. I have to add "final" onto BinaryOperator in order to
use Trailing storage. But when I do that I can't derive
CompoundAssignmentOperator from BinaryOperator. So I think I must fold these 2
classes together. Is there another way?
> -Original Message-
> From: Jo
Rumeet, Thanks for the additional info. At some point we'd like to re-commit
this patch. LNT testing showed execution time regressions on arch including
x86 and aarch. I didn't capture the test names each time but it appeared to be
approximately the same tests regressing everywhere. Andy Kay
I reverted MaskRay's "reland" since the original patch is causing trouble on
PowerPC, check-all is passing on my box. Sorry for the trouble.
> -Original Message-
> From: Andy Kaylor via Phabricator
> Sent: Thursday, February 13, 2020 9:20 PM
> To: Blower, Melanie I ; lebedev...@gmail.c