[Lldb-commits] [PATCH] D54692: [Driver] Use libOption with tablegen.

Mon Nov 26 08:57:47 PST 2018

davide added a comment.

In D54692#1308071 <https://reviews.llvm.org/D54692#1308071>, @clayborg wrote:

> Would be great to see old and new output like Zach suggested. Is there a reason we need to use TableGen? Other command line tools just use llvm:🆑:opt stuff. Seems a bit obtuse to use TableGen?

I'm afraid this is not true.  lld uses TableGen and that was a big success. See https://github.com/llvm-mirror/lld/blob/master/ELF/Options.td (considering that linkers have to handle all the annoying cmdline compatibility with gcc, e.g. options which take single or double underscores).

FWIW, I'm in favour of this change.

Repository:
  rLLDB LLDB

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54692/new/

https://reviews.llvm.org/D54692