Thanks John. Here is the fixed patch. Any other comments? Thanks Guy
-----Original Message----- From: John McCall [mailto:[email protected]] Sent: Thursday, February 07, 2013 21:50 To: Benyei, Guy Cc: [email protected] Subject: Re: [PATCH]Handle intel_ocl_bicc calling convention On Feb 6, 2013, at 6:05 AM, "Benyei, Guy" <[email protected]> wrote: > Attached a patch, which changes the ABI for functions with intel_ocl_bicc for > x86_64 non-windows targets. > This patch is needed to support the Intel OpenCL builtin library. - unsigned neededInt, neededSSE; - it->info = classifyArgumentType(it->type, freeIntRegs, neededInt, - neededSSE); + if (FI.getCallingConvention() == llvm::CallingConv::Intel_OCL_BI) + it->info = classifyOCLBIType(it->type); + else { Instead of reindenting the main loop body, please just do this at the top: if (FI.getCallingConvention() == llvm::CallingConv::Intel_OCL_BI) { it->info = classifyOCLBIType(it->type); continue; } John. --------------------------------------------------------------------- Intel Israel (74) Limited This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies.
inteloclbicc_ABI2.patch
Description: inteloclbicc_ABI2.patch
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
