<div dir="ltr">On Thu, Sep 12, 2013 at 12:29 PM, Michael Spencer <span dir="ltr"><<a href="mailto:bigcheesegs@gmail.com" target="_blank">bigcheesegs@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
================<br>
Comment at: lib/ReaderWriter/Native/NativeFileFormat.h:171-178<br>
@@ -169,1 +170,10 @@<br>
<div class="im"><br>
+//<br>
+// The NCS_UndefinedAtomsV2 chunk contains an array of these structs<br>
+//<br>
+struct NativeUndefinedAtomIvarsV2 {<br>
+  uint32_t  nameOffset;<br>
+  uint32_t  flags;<br>
+  uint32_t  fallbackNameOffset;<br>
+};<br>
+<br>
----------------<br>
</div><div class="im">Rui Ueyama wrote:<br>
> Shankar Kalpathi Easwaran wrote:<br>
> > We should just add the fallbackNameOffset to V1 (or) if you want to still do this, you could derive the second struct from the first one.<br>
> ><br>
> > struct NativeUndefinedAtomIvarsV2 : public NativeUndefinedAtomIvarsV1<br>
> done.<br>
</div>We are still modifying V1.<br></blockquote><div><br></div><div>Shankar said that we should add the field to V1? If we shouldn't we could define V2 and remove the field from V1.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


================<br>
Comment at: test/core/undef-fallback.objtxt:13-14<br>
@@ +12,4 @@<br>
+  - name: foo<br>
+    fallback:<br>
+      name: bar<br>
+...<br>
----------------<br>
Why isn't this just fallback: bar ?<br></blockquote><div><br></div><div>Because fallback is not a string but an undefined atom.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D1550" target="_blank">http://llvm-reviews.chandlerc.com/D1550</a><br>
<br>
COMMIT<br>
  <a href="http://llvm-reviews.chandlerc.com/rL190625" target="_blank">http://llvm-reviews.chandlerc.com/rL190625</a><br>
</div></div></blockquote></div><br></div></div>