[PATCH] Mangler: Don't prefix MS ABI names with a leading ? character
Rafael Ávila de Espíndola
rafael.espindola at gmail.com
Mon Mar 2 15:56:44 PST 2015
This needs at least three extra tests:
- Showing that ELF still have a .L prefix when given something like '@"?foo" private global i32 42'
- Similar for MachO, it should still get a L prefix
- A test showing the difference that now a MachO GV starting with ? will *not* get a _ prefix.
It might also be nice to have a langref/release notes update.
And last but not least, Grosbach is probably the one to approve the MachO change.
http://reviews.llvm.org/D7775
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list