[polly] r219017 - [Fix] Accidently changed the type of a libgomp argument in r219003.

Johannes Doerfert doerfert at cs.uni-saarland.de
Fri Oct 3 13:40:25 PDT 2014


Author: jdoerfert
Date: Fri Oct  3 15:40:24 2014
New Revision: 219017

URL: http://llvm.org/viewvc/llvm-project?rev=219017&view=rev
Log:
[Fix] Accidently changed the type of a libgomp argument in r219003.

  Only subsequent patches introduced tests for the signature in the
  generated IR, thus the tests were wrong too and are adjusted now.


Modified:
    polly/trunk/lib/CodeGen/LoopGenerators.cpp
    polly/trunk/test/Cloog/CodeGen/OpenMP/lifetime_markers.ll
    polly/trunk/test/Isl/CodeGen/openmp_limit_threads.ll

Modified: polly/trunk/lib/CodeGen/LoopGenerators.cpp
URL: http://llvm.org/viewvc/llvm-project/polly/trunk/lib/CodeGen/LoopGenerators.cpp?rev=219017&r1=219016&r2=219017&view=diff
==============================================================================
--- polly/trunk/lib/CodeGen/LoopGenerators.cpp (original)
+++ polly/trunk/lib/CodeGen/LoopGenerators.cpp Fri Oct  3 15:40:24 2014
@@ -189,14 +189,14 @@ void ParallelLoopGenerator::createCallSp
 
     Type *Params[] = {PointerType::getUnqual(FunctionType::get(
                           Builder.getVoidTy(), Builder.getInt8PtrTy(), false)),
-                      Builder.getInt8PtrTy(), LongType, LongType, LongType,
-                      LongType};
+                      Builder.getInt8PtrTy(), Builder.getInt32Ty(), LongType,
+                      LongType, LongType};
 
     FunctionType *Ty = FunctionType::get(Builder.getVoidTy(), Params, false);
     F = Function::Create(Ty, Linkage, Name, M);
   }
 
-  Value *NumberOfThreads = ConstantInt::get(LongType, PollyNumThreads);
+  Value *NumberOfThreads = Builder.getInt32(PollyNumThreads);
   Value *Args[] = {SubFn, SubFnParam, NumberOfThreads, LB, UB, Stride};
 
   Builder.CreateCall(F, Args);

Modified: polly/trunk/test/Cloog/CodeGen/OpenMP/lifetime_markers.ll
URL: http://llvm.org/viewvc/llvm-project/polly/trunk/test/Cloog/CodeGen/OpenMP/lifetime_markers.ll?rev=219017&r1=219016&r2=219017&view=diff
==============================================================================
--- polly/trunk/test/Cloog/CodeGen/OpenMP/lifetime_markers.ll (original)
+++ polly/trunk/test/Cloog/CodeGen/OpenMP/lifetime_markers.ll Fri Oct  3 15:40:24 2014
@@ -13,7 +13,7 @@
 ; CHECK-NEXT:    %[[GEP:[._0-9a-zA-Z]*]] = getelementptr inbounds { i32* }* %polly.par.userContext, i32 0, i32 0
 ; CHECK-NEXT:    store i32* %A, i32** %[[GEP]]
 ; CHECK-NEXT:    %polly.par.userContext{{[0-9]*}} = bitcast { i32* }* %polly.par.userContext to i8*
-; CHECK-NEXT:    call void @GOMP_parallel_loop_runtime_start(void (i8*)* @jd.polly.subfn, i8* %polly.par.userContext{{[0-9]*}}, i64 0, i64 0, i64 1024, i64 1)
+; CHECK-NEXT:    call void @GOMP_parallel_loop_runtime_start(void (i8*)* @jd.polly.subfn, i8* %polly.par.userContext{{[0-9]*}}, i32 0, i64 0, i64 1024, i64 1)
 ; CHECK-NEXT:    call void @jd.polly.subfn(i8* %polly.par.userContext{{[0-9]*}})
 ; CHECK-NEXT:    call void @GOMP_parallel_end()
 ; CHECK-NEXT:    %[[BC2:[._0-9a-zA-Z]*]] = bitcast { i32* }* %polly.par.userContext to i8*

Modified: polly/trunk/test/Isl/CodeGen/openmp_limit_threads.ll
URL: http://llvm.org/viewvc/llvm-project/polly/trunk/test/Isl/CodeGen/openmp_limit_threads.ll?rev=219017&r1=219016&r2=219017&view=diff
==============================================================================
--- polly/trunk/test/Isl/CodeGen/openmp_limit_threads.ll (original)
+++ polly/trunk/test/Isl/CodeGen/openmp_limit_threads.ll Fri Oct  3 15:40:24 2014
@@ -2,9 +2,9 @@
 ; RUN: opt %loadPolly -polly-codegen -enable-polly-openmp -polly-num-threads=1 -S < %s | FileCheck %s --check-prefix=ONE
 ; RUN: opt %loadPolly -polly-codegen -enable-polly-openmp -polly-num-threads=4 -S < %s | FileCheck %s --check-prefix=FOUR
 ;
-; AUTO: call void @GOMP_parallel_loop_runtime_start(void (i8*)* @jd.polly.subfn, i8* %polly.par.userContext{{[0-9]*}}, i64 0, i64 0, i64 1024, i64 1)
-; ONE: call void @GOMP_parallel_loop_runtime_start(void (i8*)* @jd.polly.subfn, i8* %polly.par.userContext{{[0-9]*}}, i64 1, i64 0, i64 1024, i64 1)
-; FOUR: call void @GOMP_parallel_loop_runtime_start(void (i8*)* @jd.polly.subfn, i8* %polly.par.userContext{{[0-9]*}}, i64 4, i64 0, i64 1024, i64 1)
+; AUTO: call void @GOMP_parallel_loop_runtime_start(void (i8*)* @jd.polly.subfn, i8* %polly.par.userContext{{[0-9]*}}, i32 0, i64 0, i64 1024, i64 1)
+; ONE: call void @GOMP_parallel_loop_runtime_start(void (i8*)* @jd.polly.subfn, i8* %polly.par.userContext{{[0-9]*}}, i32 1, i64 0, i64 1024, i64 1)
+; FOUR: call void @GOMP_parallel_loop_runtime_start(void (i8*)* @jd.polly.subfn, i8* %polly.par.userContext{{[0-9]*}}, i32 4, i64 0, i64 1024, i64 1)
 ;
 ;    void jd(int *A) {
 ;      for (int i = 0; i < 1024; i++)





More information about the llvm-commits mailing list