<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Dec 11, 2018 at 4:22 PM Leonard Mosescu <<a href="mailto:mosescu@google.com">mosescu@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I guess I don't see why we need a temporary solution at all.  If we can have logic that can be rolled into the SymbolVendor when we get it, and makes sense there, and is also simple, why not go with it?  Failing that, doesn't the `target symbols add` solution also work fine?<br></blockquote><div><br></div></div><div dir="ltr"><div>I just checked again, and "target symbols add" depends on having a real SymbolVendor implementation. So unfortunately we still need a temporary solution.</div></div></blockquote><div><br></div><div>I tried to verify this behavior, but it seems like it should already work out of the box?   So we're on the same page, we already do have a real SymbolVendor implementation, it just happens to be the *default* SymbolVendor implementation.  It's not the case that one doesn't exist at all.</div><div><br></div><div>So anyway, when I run "target symbols add foo.exe path/to/foo.pdb" internally what this does is set a member variable called m_symfile_spec on the Module object.  Then, it calls our normal CalculateAbilities() function which calls loadMatchingPDBFile.</div><div><br></div><div>I think all that needs to happen is that we need to check this field.  If it's empty, try to load a matching PDB file.  If it's set to something, then load that file instead.</div></div></div>