[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.



More information about the llvm-commits mailing list