[cfe-commits] [PATCH] Add multiple prefixes to Option.

Daniel Dunbar daniel at zuster.org
Fri Oct 19 10:38:26 PDT 2012


  Hi Michael,

  I understand what this patch does, but I don't really understand the
  motivation for it, yet.

  Why is this the "main feature" needed for sane gnu-ld and link command
  line support?

  It seems to me, that what you really need for those is a completely
  separate option parsing table. I can see how this patch could be
  useful if you wanted to accept the same options for the link.exe
  parsing just with a '/' at the front, but that doesn't seem right to
  me. It seems to me that the link.exe parsing or cl.exe parsing would
  end up using a completely separate parsing table.

  Can you explain?

   - Daniel

http://llvm-reviews.chandlerc.com/D69



More information about the cfe-commits mailing list