<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:Gisha;
panose-1:2 11 5 2 4 2 4 2 2 3;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:12.0pt;
font-family:"Times New Roman",serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
span.EmailStyle17
{mso-style-type:personal-reply;
font-family:"Gisha",sans-serif;
color:#4D7479;
font-weight:normal;
font-style:normal;
text-decoration:none none;}
.MsoChpDefault
{mso-style-type:export-only;
font-family:"Calibri",sans-serif;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Gisha",sans-serif;color:#4D7479">No, we just use a different mechanism now for all the other flag types (template classes in kmp_wait_release.h). Eventually 64 bit flags might be always be able
to use the same mechanism (and some of them already do), but it requires more code refactoring for cases that don’t currently use the flag templates.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Gisha",sans-serif;color:#4D7479"><o:p> </o:p></span></p>
<p class="MsoNormal"><a name="_____replyseparator"></a><b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> Yonghong Yan [mailto:yanyh15@gmail.com]
<br>
<b>Sent:</b> Tuesday, June 14, 2016 2:44 PM<br>
<b>To:</b> reviews+D21332+public+a169a41bbeffbe44@reviews.llvm.org; Peyton, Jonathan L <jonathan.l.peyton@intel.com><br>
<b>Cc:</b> Wilmarth, Terry L <terry.l.wilmarth@intel.com>; Bae, Hansang <hansang.bae@intel.com>; openmp-commits@lists.llvm.org<br>
<b>Subject:</b> Re: [Openmp-commits] [PATCH] D21332: Remove unused wait/release code.<o:p></o:p></span></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Does that mean 32-bit support is being removed? how about those *_4 and *_4u ones, e.g. __kmpc_dispatch_fini_4u? I know those __kmpc are compiler interfaces. <o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><br clear="all">
<o:p></o:p></p>
<div>
<div>
<div>
<div>
<p class="MsoNormal">If no 32-bit, how about also __kmp_wait_64 to __kmp_wait ?<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</div>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal">On Tue, Jun 14, 2016 at 3:22 PM, Jonathan Peyton via Openmp-commits <<a href="mailto:openmp-commits@lists.llvm.org" target="_blank">openmp-commits@lists.llvm.org</a>> wrote:<o:p></o:p></p>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal" style="margin-bottom:12.0pt">This revision was automatically updated to reflect the committed changes.<br>
Closed by commit rL272697: Remove unused wait/release code. (authored by jlpeyton).<br>
<br>
Changed prior to commit:<br>
<a href="http://reviews.llvm.org/D21332?vs=60714&id=60721#toc" target="_blank">
http://reviews.llvm.org/D21332?vs=60714&id=60721#toc</a><br>
<br>
Repository:<br>
rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D21332" target="_blank">http://reviews.llvm.org/D21332</a><br>
<br>
Files:<br>
openmp/trunk/runtime/src/dllexports<br>
openmp/trunk/runtime/src/exports_so.txt<br>
openmp/trunk/runtime/src/kmp.h<br>
openmp/trunk/runtime/src/kmp_wait_release.cpp<br>
<br>
<br>
_______________________________________________<br>
Openmp-commits mailing list<br>
<a href="mailto:Openmp-commits@lists.llvm.org">Openmp-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/openmp-commits" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/openmp-commits</a><o:p></o:p></p>
</blockquote>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</div>
</div>
</body>
</html>