I'm not opposed to a name change at some level or another, but our getName and getLibraryName has to return the same thing as DIA's.  Right now experimentation shows that IDiaSymbol::getName() returns the first of those two strings and in the serialized record.  "Module" and "object" just come from the fact that they use the same name.<br><br>As long as getName() returns the first field and getLibraryName() returns the second (which we have to since that's what DIA does), we can call them anything.  Alternatively we could stop using getName() and getLibraryName() entirely in PDBSymbolCompiland and call them something else, but if the api is going to mirror dia, then it should return the same values, and ideally they should either be named exactly as they are in the reference implementation (for cross verification), or completely differently <br><div class="gmail_quote"><div dir="ltr">On Tue, Mar 14, 2017 at 4:34 PM Adrian McCarthy via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">amccarth abandoned this revision.<br class="gmail_msg">
amccarth added a comment.<br class="gmail_msg">
<br class="gmail_msg">
I'm not completely convinced this is the wrong thing to do, but I won't fight it further.  I've compensated for the backwards naming in my next patch.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D30952" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D30952</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>