[PATCH] ARM: Update R_ARM_TARGET1 command line option names (remove prefix 'arm-')
Leny Kholodov
leny.kholodov at gmail.com
Thu Apr 16 05:01:41 PDT 2015
REPOSITORY
rL LLVM
================
Comment at: lib/Driver/GnuLdDriver.cpp:552
@@ -551,3 +551,3 @@
}
} else if (hasArmTarget1Rel) {
diag << "warning: ignoring unsupported ARM/ELF specific argument: "
----------------
atanasyan wrote:
> 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";
> }
> }
> ```
Thank you for this hint. I didn't know about such possibility.
http://reviews.llvm.org/D9037
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list