[PATCH] D137799: [WIP][llvm-driver] Pass extra arguments to tools

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 11 15:19:55 PST 2022


phosek added inline comments.


================
Comment at: llvm/include/llvm/Support/LLVMDriver.h:14
+
+struct LLVMDriverExtraArgs {};
+
----------------
phosek wrote:
> Since this struct will contain more than just args, like `IsDriver` added in D137800, I'd probably call it something like `LLVMDriverContext`.
Another option would be `LLVMToolContext` or even just `ToolContext` since this is already in the `llvm` namespace (we already have `ToolOutputFile` in `Support`).


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

https://reviews.llvm.org/D137799



More information about the llvm-commits mailing list