[polly] r219003 - [Refactor] Generalize parallel code generation

Johannes Doerfert doerfert at cs.uni-saarland.de
Fri Oct 3 14:03:24 PDT 2014


On 10/03, Tobias Grosser wrote:
> On 03/10/2014 21:10, Johannes Doerfert wrote:
> >-                      Builder.getInt8PtrTy(), Builder.getInt32Ty(), LongTy,
> >-                      LongTy, LongTy};
> >+                      Builder.getInt8PtrTy(), LongType, LongType, LongType,
> >+                      LongType};
> 
> Hi Johannes,
> 
> while porting my openmp patch to your code I realized you changed
> accidentally the signature of this function (The second argument is
> transformed from type Int32 to Long). This will cause a mismatch in the
> function signature between the callees in libgomp and the call we generate.
> 
> Here the original function signature:
> 
> http://rosecompiler.org/ROSE_HTML_Reference/libgomp__g_8h.html#8ddee262d7a5838878f06ccad9686668
Thanks. Fixed in r219017.

The two subsequent patches will actually cover that now, before there
was no test for it.

-- 

Johannes Doerfert
Researcher / PhD Student

Compiler Design Lab (Prof. Hack)
Saarland University, Computer Science
Building E1.3, Room 4.26

Tel. +49 (0)681 302-57521 : doerfert at cs.uni-saarland.de
Fax. +49 (0)681 302-3065  : http://www.cdl.uni-saarland.de/people/doerfert
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 213 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141003/a481f8dd/attachment.sig>


More information about the llvm-commits mailing list