<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Dec 1, 2014 at 12:11 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"><span class="">On 12/1/2014 2:04 PM, Rui Ueyama wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'm not sure if I got what you mean... So, just to make sure we are on the<br>
same page, the reason to have the round-trip passes is to make sure all<br>
output can be serialized and de-serialized, right? I think you added these<br>
tests because of that reason.<br>
<br>
Because the YAML/Native are guaranteed compatible, we can dump an<br>
intermediate results to YAML/Native files and then resume processing by<br>
reading them back.<br>
<br>
And then looks like you are now adding a way to bypass that. So they are no<br>
longer compatible. That is exactly what you wanted to avoid, no?<br>
</blockquote></span>
Yes, I thought of adding a way to bypass that but you are right, we should not allow the RoundTripPasses to be bypassed. I will remove the setRunRoundTripPass function in LinkingContext.<br>
<br>
But I do need lld to preserve the filename of the atom using references if the round trip passes were going to be run. So the reader and the writer would read the information using references if the round trip passes were being run.</blockquote><div><br></div><div>If you need a new property, you can add that to Atom. (Code review would be needed though.)</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888"><br>
<br>
Shankar Easwaran</font></span><div class="HOEnZb"><div class="h5"><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></div>