[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