[PATCH] D49930: [DebugInfo][OpenCL] Generate correct block literal debug info for OpenCL

Eric Christopher via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 30 13:11:52 PDT 2018


echristo added a comment.

The patch is fine, in general, couple of comments:

a) Can you refactor this so the if conditionals are just two functions? Those functions are big enough already.
b) I'm not quite sure why you're picking limited here, do you have an explanation?
c) Can you split that part out into a separate test? Additional run lines in the existing blocks.cl test would be fine.

Thanks!


Repository:
  rC Clang

https://reviews.llvm.org/D49930





More information about the cfe-commits mailing list