[PATCH] D102094: [AIX][PowerPC] Remove error when specifying mabi=vec-default on AIX

Zarko Todorovski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 10 09:01:15 PDT 2021


ZarkoCA marked an inline comment as done.
ZarkoCA added inline comments.


================
Comment at: clang/lib/Frontend/CompilerInvocation.cpp:1854
     const Option &O = A->getOption();
-    if (O.matches(OPT_mabi_EQ_vec_default))
-      Diags.Report(diag::err_aix_default_altivec_abi)
-          << A->getSpelling() << T.str();
-    else {
-      assert(O.matches(OPT_mabi_EQ_vec_extabi));
+    if (O.matches(OPT_mabi_EQ_vec_extabi))
       Opts.EnableAIXExtendedAltivecABI = 1;
----------------
jansvoboda11 wrote:
> Can we simplify this to `Opts.EnableAIXExtendedAltivecABI = O.matches(OPT_mabi_EQ_vec_extabi);`?
Thanks, that works well. 


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

https://reviews.llvm.org/D102094



More information about the cfe-commits mailing list