Re: [PATCH] D17955: [OpenCL] Fix piep builtin bug

2016-03-08 Thread Mats Petersson via cfe-commits
MatsPetersson added a subscriber: MatsPetersson. MatsPetersson added a comment. Obvious typo in title. Pipe, not piep. http://reviews.llvm.org/D17955 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listi

[PATCH] D17955: [OpenCL] Fix piep builtin bug

2016-03-08 Thread Xiuli PAN via cfe-commits
pxli168 created this revision. pxli168 added reviewers: Anastasia, yaxunl. pxli168 added subscribers: cfe-commits, pekka.jaaskelainen. 1. Diag should be output if types are not the same. 2. Should compare using canonical type. 3. Refine the diag to be more clear. http://reviews.llvm.org/D17955 F