<div dir="ltr">We don't need a filename for the PE/COFF writer, but if it existed, it wouldn't hurt us. We'll leave the field nullptr.<div><br></div><div>I don't think we need to vote here. Even if only one arch needs it, if it should naturally be added to Atom, it should be added to Atom.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 1, 2014 at 12:44 PM, Shankar Easwaran <span dir="ltr"><<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+ Nick<br>
<br>
Rui,<br>
<br>
Does PECOFF writer need the filename in the writer as well, I am not sure if linker scripts are supported with PECOFF though.<br>
<br>
If PECOFF also needs it, I think it makes sense to store the filename in the Atom as the native format needs to store that information.<br>
<br>
The only option for the ELF writer to know this information is to use References if other flavors dont need the filename (only in DEBUG mode, clumsy but would work).<br>
<br>
PS : Moving this discussion to llvmdev.<br>
<br>
Shankar Easwaran<br>
<br>
On 12/1/2014 2:34 PM, Rui Ueyama wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
That sounds like we really need a new property of Atom.<br>
<br>
1. If we run LLD in Release build, the roundtrip passes don't run, so<br>
everything works fine.<br>
2. If we run LLD in Debug build (and from the unit tests), the information<br>
is dropped during the round-trip conversion, and it fails.<br>
3. RoundTrip tests should't drop any information.<br>
<br>
2 and 3 conflicts.<br>
<br>
I should note that, again, I don't actually like the idea of YAML/Native<br>
format, though. It feels like it doesn't worth the cost of maintaining two<br>
more different outputs.<br>
<br>
On Mon, Dec 1, 2014 at 12:20 PM, Shankar Easwaran <<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.org</a>><br>
wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Rui,<br>
<br>
We discussed to add a new property to the atom, but its really not needed<br>
as the original filename from where the atom was parsed is available in<br>
release mode(roundtrip passes dont get called in release mode).<br>
<br>
The discussion was here, <a href="http://lists.cs.uiuc.edu/pipermail/llvmdev/2014-" target="_blank">http://lists.cs.uiuc.edu/<u></u>pipermail/llvmdev/2014-</a><br>
November/078910.html.<br>
<br>
Shankar Easwaran<br>
<br>
<br>
<br><span class="HOEnZb"><font color="#888888">
</font></span></blockquote></blockquote><span class="HOEnZb"><font color="#888888">
<br>
-- <br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation<br>
<br>
</font></span></blockquote></div><br></div>