[PATCH] D127472: [llvm-driver] Add lld

[Alex Brachet via Phabricator via llvm-commits]

abrachet added inline comments.


================
Comment at: lld/include/lld/Common/Driver.h:28
 // by cleanup.
-SafeReturn safeLldMain(int argc, const char **argv, llvm::raw_ostream &stdoutOS,
+SafeReturn safeLldMain(int argc, char **argv, llvm::raw_ostream &stdoutOS,
                        llvm::raw_ostream &stderrOS);
----------------
MaskRay wrote:
> Seem unnecessary?
I don't understand why, but `T**` doesn't implicitly convert to `const T **`, and the only way I could find to add the `const` qualifier to the inner pointer was through `const_cast` so I can either keep this as `const char **` and `const_cast` in from `lld_main` or keep this as `char **`. WDYT?


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

https://reviews.llvm.org/D127472