[llvm] r259675 - Revert r259662, which caused regressions on polly tests.

Tobias Grosser via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 3 21:59:42 PST 2016


On 02/04/2016 12:37 AM, Hal Finkel wrote:
> ----- Original Message -----
>> From: "Wei Mi" <wmi at google.com>
>> To: "Hal Finkel" <hfinkel at anl.gov>
>> Cc: "Tobias Grosser" <tobias at grosser.es>, "llvm-commits" <llvm-commits at lists.llvm.org>
>> Sent: Wednesday, February 3, 2016 5:31:13 PM
>> Subject: Re: [llvm] r259675 - Revert r259662, which caused regressions on polly tests.
>>
>>> 1) test/Isl/CodeGen/non-affine-phi-node-expansion.ll
>>
>>> In this test case LLVM after your patch actually produces more
>>> complicated IR then before. Specifically, it does not inline the
>>> 'true'
>>> value any more. This is clearly a minor regression which will be
>>> cleaned up by later passes, but it might be of interest to you
>
> Sounds good. As far as I'm concerned, you can re-commit with the fixes (please make sure the explanations from these e-mails make it into the commit message along with the original text).

Perfect, Wei Mi. This works for me. Please feel free to commit the 
necessary Polly test case changes as well.

Best,
Tobias



More information about the llvm-commits mailing list