<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p>By the way, seems to me, I won't be able to do it today, because
I have some troubles with accessing our server. I will do it
tomorrow when I get the access back.<br>
</p>
<pre class="moz-signature" cols="72">-------------
Best regards,
Alexey Bataev</pre>
<div class="moz-cite-prefix">28.02.2018 16:08, Alexey Bataev пишет:<br>
</div>
<blockquote type="cite"
cite="mid:AM5PR0902MB1923F77B8EF1FB385D58CECC85C70@AM5PR0902MB1923.eurprd09.prod.outlook.com">
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<pre class="moz-signature" cols="72">Unfortunately, I can't commit everything else without this patch. It will definitely break the compilation of ptx files.
-------------
Best regards,
Alexey Bataev</pre>
<div class="moz-cite-prefix">28.02.2018 16:04, Eric Christopher
пишет:<br>
</div>
<blockquote type="cite"
cite="mid:CALehDX4-CfuCUMbZDz9wzMPmUCQBfFtdMP2jj0jNjbqfirOwJA@mail.gmail.com">
<meta http-equiv="Content-Type" content="text/html;
charset=utf-8">
<div dir="ltr"><br>
<br>
<div class="gmail_quote">
<div dir="ltr">On Wed, Feb 28, 2018 at 1:02 PM Alexey Bataev
via Phabricator <<a
href="mailto:reviews@reviews.llvm.org"
moz-do-not-send="true">reviews@reviews.llvm.org</a>>
wrote:<br>
</div>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">ABataev
added a comment.<br>
<br>
In <a
href="https://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Freviews.llvm.org%2FD43627%231022609&data=02%7C01%7C%7C7bbf5fc8944e49fc542008d57eeeed99%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636554487011067680&sdata=kd5OVnYezYu1jsqgji8K03%2FYJYvIpDWZiRCmL2cBiwQ%3D&reserved=0"
originalsrc="https://reviews.llvm.org/D43627#1022609"
shash="bYlz50mlETSeBw+5jGJgxBKGHQmGENEiAP5iZNYgJzkRzztCU5ezfiMd9scZGWJGLfXkU6cbb9i+s4AMOtmUsh7obRGuXi+F/ear0OZXUEbbemXuQXLqm1VtuX4BlMZg9ZG44+kc1D6qN0sQm5Rzxpy1rG3YeorwU3XPrBy65eA="
rel="noreferrer" target="_blank" moz-do-not-send="true">https://reviews.llvm.org/D43627#1022609</a>,
@echristo wrote:<br>
<br>
> Using sections as references is, in general, a good
idea when we can so I'd rather not tie that to whether or
not we emit certain sections. If you don't mind reverting
that aspect of the change I'd appreciate it and we can get
that in via some other preference or a strict requirement
via the asm printer as we're planning on doing for the
rest of the nvptx section emission.<br>
<br>
<br>
These sections have labels inside of them + they are not
supported by PTX format. Ok, I will revert the patch.<br>
<br>
</blockquote>
<div><br>
</div>
<div>Sure. Adding back everything else is fine to commit
btw. We can worry about when to use section relocations
rather than label relocations in a different patch though.</div>
<div><br>
</div>
<div>-eric </div>
</div>
</div>
</blockquote>
<br>
</blockquote>
<br>
</body>
</html>