[polly] r271538 - Temporarily promote values to i64 again

Tobias Grosser via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 7 09:38:28 PDT 2016


On 06/02/2016 07:09 PM, Johannes Doerfert via llvm-commits wrote:
> Author: jdoerfert
> Date: Thu Jun  2 12:09:22 2016
> New Revision: 271538
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=271538&view=rev
> Log:
> Temporarily promote values to i64 again
> 
>   Operands of binary operations that might overflow will be temporarily
>   promoted to i64 again, though that is not a sound solution for the problem.

This is probably one issue that was introduced by the 271513. After this
fix it is still unclear to me if 271513 is now NFC or not and if all
potential issues have been resolved. As suggested earlier, due to the
fundamental issues you touch it is probably better to revert the code
and then discuss it first.

Best,
Tobias


More information about the llvm-commits mailing list