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

Jan Svoboda via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 10 00:10:32 PDT 2021


jansvoboda11 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;
----------------
Can we simplify this to `Opts.EnableAIXExtendedAltivecABI = O.matches(OPT_mabi_EQ_vec_extabi);`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102094



More information about the cfe-commits mailing list