Call CalculateSymbolSizes in ObjectFilePECOFF::GetSymtab

[amccarth]

Summary

This fixes the nine tests that regressed on Windows after http://reviews.llvm.org/D16186

[zturner]

looks fine to me, but some Apple targets use PE, so +greg in case he can think of a reason why this will mess up something on the Apple side.

[tberghammer]

Looks good to me and if Apple using this code then I am pretty sure my CL break them as well. Having 0 size symbols around will cause a lot of issue around LLDB.

[clayborg]

I just added PECOFF so I could make sure I had 3 clients for the ObjectFile class when I was developing it and I already have the PECOFF support in the previous tool I had written, so this is quite fine to do. I could even fine a PECOFF file that had a symbol table when I made ObjectFilePECOFF so I wasn't able to actually use any of the symbols. We don't use PECOFF at Apple, so this looks good.

source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp

if (module_sp)
 
                     Address symbol_addr(m_coff_header_opt.image_base + function_rva, sect_list);
                     symbols[i].GetMangled().SetValue(ConstString(symbol_name.c_str()));
                     symbols[i].GetAddressRef() = symbol_addr;
                     symbols[i].SetType(lldb::eSymbolTypeCode);
                     symbols[i].SetDebug(true);
                 }
             }
+            m_symtab_ap->CalculateSymbolSizes();
         }
     }
     return m_symtab_ap.get();
 
 }
 
 bool
 ObjectFilePECOFF::IsStripped ()