<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body>
<pre class="moz-signature" cols="72">There is no support for no ', debug' in this patch, I'm going to implement it in the next patch, NVPTX specific. This one is a generic.
-------------
Best regards,
Alexey Bataev</pre>
<div class="moz-cite-prefix">24.04.2018 13:11, David Blaikie пишет:<br>
</div>
<blockquote type="cite" cite="mid:CAENS6EvZLvo0FwUth4Lmhc3uG=mcYURaK2RuT8yawYFM_DT0wQ@mail.gmail.com">
<div dir="ltr"><br>
<br>
<div class="gmail_quote">
<div dir="ltr">On Tue, Apr 24, 2018 at 10:08 AM 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://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Freviews.llvm.org%2FD46021%231077023&data=02%7C01%7C%7C9f1f4b4c119146a32ac108d5aa06689f%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636601866850650990&sdata=w7spmo1u8yJbBS6DD2H%2BUrgRoxqVC9eQ8x20sQEHP4A%3D&reserved=0" originalsrc="https://reviews.llvm.org/D46021#1077023" shash="pRf0uINspNPGw1VN55t4pugAGtEVVDxxMk14XQdZphDoFFs0m0kLH2VxXnhTVo1yIa39rewS7NFniOWAq1vjP5qguz7DtN6SRamWmrShGRXgmlHLq7IWqJiJ9N8ZU5DBFwGKiH7dUSSkWvkRkjom1Q7W7fcay58/xQ/PSjOv0Bs=" rel="noreferrer" target="_blank" moz-do-not-send="true">
https://reviews.llvm.org/D46021#1077023</a>, @dblaikie wrote:<br>
<br>
> Sorry, I'm a bit confused.<br>
><br>
> 1. I'd expect /this/ behavior (line table directives, but no debug_info, etc) to be controlled via a DICompileUnit emission kind (a value between "NoDebug" and "LineTablesOnly" as Paul mentioned).<br>
> 2. I'd expect the "no ', debug'" flag (which I don't see in this change? Is it missing? is it elsewhere?) to be controlled via an MCOption - because it must be global.<br>
><br>
> Now, arguably, if we have the global "no ', debug'" flag we maybe might as well use it to implement this behavior too since we don't have any other use for it? So maybe (2) invalidates (1).<br>
<br>
<br>
I thought, that it is not very useful to have an option that, actually, controls only single target and I decided instead to add the option that just disables emission of the DWARF sections. In this case we don't need the special debug info mode, we can use
this option.<br>
</blockquote>
<div><br>
I'm not sure I follow - specifically I'm still not sure where the "no ', debug'" support is in this patch - did I miss it?<br>
<br>
As for whether it's useful to have the "don't emit debug sections except for the line table" as a DICompileUnit feature - the reason to put it there would be for consistency with other related variations in debug info emission, even if it is only used by one
target.<br>
</div>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
Repository:<br>
rL LLVM<br>
<br>
<a href="https://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Freviews.llvm.org%2FD46021&data=02%7C01%7C%7C9f1f4b4c119146a32ac108d5aa06689f%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636601866850650990&sdata=4dmAKbJQOcfYKnRKFL7qpae5jB%2BNhNwfuXIFs7CoI2o%3D&reserved=0" originalsrc="https://reviews.llvm.org/D46021" shash="lWqTjx3draDa50nJRd6mcVcCOPUn9jLBoj9jPaDj3REwyXMEBAmUrAYcxM7OhXs9ZFlD1dWoUaahrianNBhZgOaVpZfJftzqpBFirJPPhasqyNdqjTBxShPatsw4P3BWCyabcmeO4Eu4qCwOCFPFvoDaWbZaiZrh+NnuA0QriYg=" rel="noreferrer" target="_blank" moz-do-not-send="true">https://reviews.llvm.org/D46021</a><br>
<br>
<br>
<br>
</blockquote>
</div>
</div>
</blockquote>
<br>
</body>
</html>