Command line options --arm-target1-rel and --arm-target1-abs have been renamed to be compatible with GNU linkers.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
LGTM
lib/Driver/GnuLdDriver.cpp | ||
---|---|---|
549 ↗ | (On Diff #23795) | This does not relate to this patch but the else if statement violates the "else after a return" rule: |
552 ↗ | (On Diff #23795) | We can handle both else if statements below in a uniform way: else { for (const auto *arg : parsedArgs->filtered(OPT_grp_arm_targetopts)) { diag << "warning: ignoring unsupported ARM/ELF specific argument: " << arg->getSpelling() << "\n"; } } |
lib/Driver/GnuLdDriver.cpp | ||
---|---|---|
552 ↗ | (On Diff #23795) | Thank you for this hint. I didn't know about such possibility. |