[PATCH] D60196: [llvm-symbolizer] Add grouping for -e

Igor Kudrin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 3 07:08:25 PDT 2019


ikudrin added a comment.

In D60196#1453050 <https://reviews.llvm.org/D60196#1453050>, @jhenderson wrote:

> Do you think it's worth testing the behaviour of -e in the middle of a group of options e.g. "-Cefi"?


It's a negative scenario, right? Haven't we already checked that in CommandLineTest.GroupingWithValue <https://github.com/llvm/llvm-project/blob/ee737a84d75b701c2ae3a8c9a0abbae63fc330af/llvm/unittests/Support/CommandLineTest.cpp#L1161>?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60196





More information about the llvm-commits mailing list