Details
Diff Detail
Event Timeline
llvm/cmake/modules/llvm-driver-template.cpp.in | ||
---|---|---|
12–14 | Since we're no longer trying to match the signature of main, we could also consider other changes, for example passing args as a ArrayRef<const char *>, but that could be done in some future changes. | |
llvm/include/llvm/Support/LLVMDriver.h | ||
15 | Since this struct will contain more than just args, like IsDriver added in D137800, I'd probably call it something like LLVMDriverContext. |
llvm/include/llvm/Support/LLVMDriver.h | ||
---|---|---|
15 | Another option would be LLVMToolContext or even just ToolContext since this is already in the llvm namespace (we already have ToolOutputFile in Support). |
llvm/cmake/modules/llvm-driver-template.cpp.in | ||
---|---|---|
12–14 |
I think I'll do that in the future because that would be a pretty involved change to the tools respective main functions. |
llvm/tools/dsymutil/CMakeLists.txt | ||
---|---|---|
43 | @abrachet It seems that on some platforms, we need to link dsymutil.cpp with libatomic. |
Since we're no longer trying to match the signature of main, we could also consider other changes, for example passing args as a ArrayRef<const char *>, but that could be done in some future changes.