<div dir="ltr">I think that's arguable if that's considered as information lost. After an atom is written to a new Native file and reloaded, it gets a new file name because it was actually read from a different file. If a compiler outputs an object file in Native format, it's the file name of an atom read from the file.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 1, 2014 at 2:14 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"><div class="HOEnZb"><div class="h5">On 12/1/2014 4:02 PM, Rui Ueyama wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Mon, Dec 1, 2014 at 1:48 PM, Nick Kledzik <<a href="mailto:kledzik@apple.com" target="_blank">kledzik@apple.com</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Dec 1, 2014, at 12:34 PM, Rui Ueyama <<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>> wrote:<br>
<br>
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>
<br>
I have a different view.<br>
<br>
I think of those Passes as a quick way to exercise the YAML and native<br>
formats.  The mechanism is as if you went through the test cases and added<br>
a line which ran “ld -r” to merge all input files into one file, then ran<br>
the rest of the test with that merged input file.  That works everywhere<br>
except in tests that care about the name of the input file (which is now<br>
changed).  So, for those cases you would not do the merge step.  But since<br>
the round trip is implicit, there is no way to turn it off.  Shankar is<br>
adding a way to turn it off.<br>
<br>
</blockquote>
That doesn't sounds like a different view.<br>
<br>
If a test fails because of the difference of an input file name, not<br>
because the round-trip conversion drops some information, we should disable<br>
the round-trip test only for that test. Now I feel I understood your<br>
suggestion in a different thread. Is there any way to override the<br>
environment variable in the lit test? Does it allow us to write RUN:<br>
LLD_ROUNDTRIP_TEST=0 lld <args here> ?<br>
</blockquote>
<br></div></div>
The filename information of the Atom is lost in the Native format.<br>
<br>
Gnu flavor needs the filename whenever the user uses a linker script (or) the user decides to know the memory map of the image.<br>
<br>
So this is needed almost all the time.<div class="HOEnZb"><div class="h5"><br>
<br>
Shankar Easwaran<br>
<br>
-- <br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation<br>
<br>
</div></div></blockquote></div><br></div>