[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