<div dir="ltr"><div>It's hard to be sure this is correct since (unless I'm misreading) this code would all be dead right now, since we don't yet support parsing this directive.</div><div><br></div><div>I'd like to see parsing support and testcases before I'd consider this ready to commit. I'm also a bit concerned that (from a skim of the OpenMP documentation) this seems to be an attribute that can be applied to functions and variables, and not a statement, so modeling it as an OMPExecutableDirective might be incorrect.</div>
<div><br></div><div><br></div>+case OMPD_declare_update: {<div><br></div><div>This line is underindented.<div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 1, 2014 at 9:19 AM, "C. Bergström" <span dir="ltr"><<a href="mailto:cbergstrom@pathscale.com" target="_blank">cbergstrom@pathscale.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi<br>
<br>
Trivial patch which implements OMP4 target declare. Can someone please review and push if is acceptable or let me know what needs fixing.<br>
<br>
@Intel - I see you guys are missing it as well.. hopefully it helps save you a few minutes.<br>
<br>
Thanks<br>
<br>
<br>_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
<br></blockquote></div><br></div></div></div>