[PATCH] [Polly][Refactor] Generalize parallel code generation
    Tobias Grosser 
    tobias at grosser.es
       
    Fri Oct  3 10:35:15 PDT 2014
    
    
  
I like the idea with the DataLayout. This makes the behaviour a lot more clear. Thanks for insisting here.
You also sneaked in a semantic change without a test case (actually it can not be tested as the current CLooG code generator always generates matching types making this basically dead code). I would feel more comfortable if we remove such asserts only when needed and with a specific test case and explaining why truncating is OK, instead of having this part of a larger refactoring.
	 if (LB->getType() != LongType)
	 LB = Builder.CreateSExtOrTrunc(LB, LongType);
	 if (UB->getType() != LongType)
	 UB = Builder.CreateSExtOrTrunc(UB, LongType);
	 if (Stride->getType() != LongType)
	 Stride = Builder.CreateSExtOrTrunc(Stride, LongType);
Otherwise LGTM.
http://reviews.llvm.org/D4990
    
    
More information about the llvm-commits
mailing list