[PATCH] D110900: Triple: Add RedHat vendor

Aaron Puchert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 2 13:27:00 PDT 2023


aaronpuchert accepted this revision.
aaronpuchert added a comment.
This revision is now accepted and ready to land.

Looks fine to me, but maybe you can wait a bit for the other reviewers in case there are still concerns.



================
Comment at: llvm/include/llvm/TargetParser/Triple.h:568-578
+    case Triple::GNU:
+    case Triple::GNUABIN32:
+    case Triple::GNUABI64:
+    case Triple::GNUEABI:
+    case Triple::GNUEABIHF:
+    case Triple::GNUF32:
+    case Triple::GNUF64:
----------------
You might use the opportunity to drop the redundant `Triple::`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110900



More information about the llvm-commits mailing list