Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
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 | ||
| 14 | 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 | ||
|---|---|---|
| 14 | 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 | ||
|---|---|---|
| 44 | @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.