This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH][IA64] Fix control speculation bug


Maxim Kuvyrkov wrote:
Hello,

This patch fixes ICE on virtually any case of using ia64 control speculation. The bug doesn't manifest itself because control speculation by default is off.

I don't have access to ia64 box at the moment, so I only checked it on a cross-compiler. Andrey, can you, possibly, test the patch?

The patch is fairly simple and only constraints the scheduler to use control speculation in lesser number of cases.

OK for mainline and 4.3 branch?


Yes, the patch itself is OK. But you should add also a test to check the control speculation for ia64. I think it is easy to do because control speculation is completely broken now.

Thanks for the patch.



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]