[Lldb-commits] [PATCH] D54682: [Driver] Extract option parsing and option processing.

Zachary Turner via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Sun Nov 18 20:19:42 PST 2018


zturner added a comment.

Lib option definitely does, as its entire purpose is to be powerful and
flexible enough to mimic arbitrary command line tools.

cl::opt is a little less powerful but it still preserves order among
multiple options with the same flag, just not multiple options with
different flags.

I don’t have a strong opinion that this particular change should be gated
on that, it’s just nice when we can reuse llvm logic. Sometimes it fixes
latent bugs, sometimes it enables new functionality that wasn’t possible
before, etc. up to you if you wanna give it a shot


Repository:
  rLLDB LLDB

https://reviews.llvm.org/D54682





More information about the lldb-commits mailing list