[polly] r267559 - Allow unsigned comparisons

Johannes Doerfert via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 27 05:56:12 PDT 2016


On 04/27, Tobias Grosser wrote:
> On 04/26/2016 04:33 PM, Johannes Doerfert via llvm-commits wrote:
> > Author: jdoerfert
> > Date: Tue Apr 26 09:33:12 2016
> > New Revision: 267559
> > 
> > URL: http://llvm.org/viewvc/llvm-project?rev=267559&view=rev
> > Log:
> > Allow unsigned comparisons
> > 
> >   With this patch we will optimistically assume that the result of an unsigned
> >   comparison is the same as the result of the same comparison interpreted as
> >   signed.
> 
> After this commit, I see a miscompile in
> http://lab.llvm.org:8011/builders/perf-x86_64-penryn-O3-polly-before-vectorizer-unprofitable/builds/1716
Thank you for noticing.

  1) I cannot reproduce this locally, queens works fine on my machine.
  2) I just pushed a fixed that fixes a problem we currently have but we
     have to see if it is related.
  3) Can anybody else reproduce the problem locally?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 213 bytes
Desc: Digital signature
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160427/1999b220/attachment.sig>


More information about the llvm-commits mailing list