<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
On Aug 20, 2020, at 1:40 AM, Kiran Chandramohan <<a href="mailto:Kiran.Chandramohan@arm.com" class="">Kiran.Chandramohan@arm.com</a>> wrote:
<div class=""><br class="">
<div>
<blockquote type="cite" class="">
<div class="">
<div style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class="">
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt;" class="">
<span style="color: rgb(32, 31, 30); font-family: Calibri, sans-serif; font-size: 14.6667px; text-align: left; background-color: rgb(255, 255, 255); display: inline !important;" class="">Hi all,<br class="">
<br class="">
I am proposing two minor amendments to the Pull Request Validation process.<span class="Apple-converted-space"> </span><br class="">
<br class="">
1) Currently, for merging a Pull Request two volunteers are required to run the tests and approve. But this has to be done for three platforms. We are finding that this requirement for three platforms is slowing down the processing since a gatekeeper do not
have access to all three platforms. I propose that we change the requirement to atleast two of Arm, X86, OpenPower.<br class="">
</span></div>
</div>
</div>
</blockquote>
<div><br class="">
</div>
I'm OK with this as long as one of the two platforms is OpenPower. :-)</div>
<div><br class="">
</div>
<div>The OpenPower builds don't get exercised as much as the Arm and X86 builds seem to, as there is no commercial compiler that is dependent on OpenPower. But OpenPower is an important platform for the flang community, so I want to make sure that pull requests
are validated on it.</div>
<div><br class="">
</div>
<div>For example, I've just started to investigate troubles with building OpenPower on clang, which doesn't seem to be an issue with Arm and X86. (It may be nothing important, but my initial attempts didn't build correctly.)</div>
<div><br class="">
</div>
<div><br class="">
<blockquote type="cite" class="">
<div class="">
<div style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class="">
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt;" class="">
<span style="color: rgb(32, 31, 30); font-family: Calibri, sans-serif; font-size: 14.6667px; text-align: left; background-color: rgb(255, 255, 255); display: inline !important;" class="">2) Currently, the process requires testing on LLVM 9.0. But we are currently
setting up support for LLVM 10.0 and vendors using Flang are moving to 10.0/11.0. I propose that we change the requirement to LLVM 9.0 or later versions supported by Flang.<br class="">
</span></div>
</div>
</div>
</blockquote>
<div><br class="">
</div>
I don't have a problem with this. I will try to build on LLVM 9 and the later releases as it's easy for me to do it once each build is working.</div>
<div><br class="">
</div>
<div><br class="">
<blockquote type="cite" class="">
<div class="">
<div style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class="">
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt;" class="">
<span style="color: rgb(32, 31, 30); font-family: Calibri, sans-serif; font-size: 14.6667px; text-align: left; background-color: rgb(255, 255, 255); display: inline !important;" class="">The proposed changes will convert the following point,<br class="">
<br class="">
Even<span class=""> </span></span><span data-markjs="true" class="mark2toj7x05b" data-ogacdata-ogabdata-ogscdata-ogsb="" style="margin: 0px; font-size: 14.6667px; font-family: Calibri, sans-serif; color: rgb(32, 31, 30); text-align: left; background-color: rgb(255, 255, 255);">for</span><span style="color: rgb(32, 31, 30); font-family: Calibri, sans-serif; font-size: 14.6667px; text-align: left; background-color: rgb(255, 255, 255); display: inline !important;" class=""><span class=""> </span>the
most trivial patch, at least two volunteers must apply the patch, run their internal tests, and give feedback on the patch. The patch should be run on at least Arm, x86 and OpenPOWER on LLVM 9.0.<br class="">
<br class="">
to</span></div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt;" class="">
<span style="color: rgb(32, 31, 30); font-family: Calibri, sans-serif; font-size: 14.6667px; text-align: left; background-color: rgb(255, 255, 255); display: inline !important;" class=""><br class="">
</span></div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt;" class="">
<span style="color: rgb(32, 31, 30); font-family: Calibri, sans-serif; font-size: 14.6667px; text-align: left; background-color: rgb(255, 255, 255); display: inline !important;" class=""><span style="margin: 0px; font-family: Calibri, sans-serif; background-color: rgb(255, 255, 255); display: inline !important;" class="">Even<span style="margin: 0px;" class=""> </span></span><span data-markjs="true" class="mark2toj7x05b" data-ogacdata-ogabdata-ogscdata-ogsb="" style="margin: 0px; font-family: Calibri, sans-serif; background-color: rgb(255, 255, 255);">for</span><span style="margin: 0px; font-family: Calibri, sans-serif; background-color: rgb(255, 255, 255); display: inline !important;" class=""><span style="margin: 0px;" class=""> </span>the
most trivial patch, at least two volunteers must apply the patch, run their internal tests, and give feedback on the patch. The patch should be run on at least<span class="Apple-converted-space"> </span><b class="">two of</b><span class="Apple-converted-space"> </span>Arm,
X86, OpenPower on LLVM<span class="Apple-converted-space"> </span><b class="">(9.0 or later versions supported by flang).</b></span></span></div>
</div>
</div>
</blockquote>
<div><br class="">
</div>
I'd just delete the "<b class="">two of</b>", for the reasons above.</div>
<div><br class="">
</div>
<div><br class="">
<blockquote type="cite" class="">
<div class="">
<div style="caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class="">
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt;" class="">
<span style="color: rgb(32, 31, 30); font-family: Calibri, sans-serif; font-size: 14.6667px; text-align: left; background-color: rgb(255, 255, 255); display: inline !important;" class="">We will discuss these two points in the Classic Flang Technical call on
Wednesday. Meeting link below. If you need a meeting invite then please let me know.<br class="">
<span style="font-family: Times; font-size: 14px; text-align: start; display: inline !important;" class=""><a href="https://teams.microsoft.com/l/meetup-join/19%3ameeting_YjM5YjlmNGEtMjA0MS00MTRlLTg5ZjUtOTM1ZGIxOTU2NWQy%40thread.v2/0?context=%7b%22Tid%22%3a%22f34e5979-57d9-4aaa-ad4d-b122a662184d%22%2c%22Oid%22%3a%223641875c-ef5b-4767-8105-0787a195852f%22%7d" id="LPlnk608215" class="">https://teams.microsoft.com/l/meetup-join/19%3ameeting_YjM5YjlmNGEtMjA0MS00MTRlLTg5ZjUtOTM1ZGIxOTU2NWQy%40thread.v2/0?context=%7b%22Tid%22%3a%22f34e5979-57d9-4aaa-ad4d-b122a662184d%22%2c%22Oid%22%3a%223641875c-ef5b-4767-8105-0787a195852f%22%7d</a></span><br class="">
<br class="">
</span><br class="">
Thanks,<br class="">
Kiran</div>
<div id="appendonsend" class=""></div>
</div>
</div>
</blockquote>
</div>
<br class="">
</div>
</body>
</html>