<div dir="ltr">Hi, I submitted this patch ~20 days ago. Could this get some comments/review if the solution is acceptable or if there is some other preferred way to implement it?<div><br></div><div>Thanks, Mikael</div></div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Sep 16, 2013 at 8:37 AM, Mikael Lepistö <span dir="ltr"><<a href="mailto:mikael.lepisto@vincit.fi" target="_blank">mikael.lepisto@vincit.fi</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi, ping</div><div><br></div><div>I sent this simple patch some time ago to list, but it got no responses. Is there any estimates, when this might be reviewed or added to main repo?</div>
<div><br></div>
<div>Thanks, Mikael Lepistö</div><div><br></div><br><div class="gmail_extra">I also added missing [PATCH] tag to subject (I forgot to to put it to original mail).<br><br><div class="gmail_quote">On Sat, Sep 7, 2013 at 12:49 PM, Mikael Lepistö <span dir="ltr"><<a href="mailto:mikael.lepisto@vincit.fi" target="_blank">mikael.lepisto@vincit.fi</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>We needed in our project (<a href="https://github.com/KhronosGroup/webcl-validator" target="_blank">https://github.com/KhronosGroup/webcl-validator</a>) to be able to define callbacks for handling OpenCL extension pragmas to catch easily which OpenCL extensions are enabled/disabled.</div>


<div><br></div><div>I attached the patch file and test case for the functionality. Rami Ylimäki is actually author of the implementation I just wrote the test case and cleaned up the patch for upstreaming.</div><div><br>

</div>
<div>I had to add clangParse and clangSema libraries to PPCallbacksTest compilation, because enabling OpenCL support for preprocessor required Parser instance. Parser instance actually reads LangOptions from preprocessor and registers correct pragma handlers to preprocessor.</div>


<div><br></div><div><div>Any comments, improvements?</div></div><div><br></div><div>Cheers, Mikael Lepistö</div><div><br></div><div><br></div></div>
</blockquote></div><br></div></div>
</blockquote></div><br></div>