[PATCH] Defining a new API for debug options that doesn't rely on static global cl::opts.

Chris Bieneman beanz at apple.com
Wed Oct 15 14:45:12 PDT 2014


This update contains changes based on chandlerc's feedback.

Changes in these patches:
* Moved the OptionKey into a detail namespace
* Updated comments based on chandlerc's feedback

Since the API seems stabilized and the feedback at this point is mostly documentation can we transition this to post-commit review for further feedback?

http://reviews.llvm.org/D5389

Files:
  include/llvm/IR/LLVMContext.h
  include/llvm/PassSupport.h
  include/llvm/Support/Options.h
  lib/Support/CMakeLists.txt
  lib/Support/CommandLine.cpp
  lib/Support/Options.cpp
  lib/Transforms/Scalar/Scalarizer.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5389.14963.patch
Type: text/x-patch
Size: 10955 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141015/9658ba6e/attachment.bin>


More information about the llvm-commits mailing list