[PATCH] D100611: [Clang] Add clang attribute `clang_builtin_alias`.

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 13 20:39:36 PDT 2021


rnk added subscribers: vsk, rnk.
rnk added inline comments.
Herald added a subscriber: jeroen.dobbelaere.


================
Comment at: clang/lib/Sema/SemaDeclAttr.cpp:5155
+#define BUILTIN(ID, TYPE, ATTRS) case RISCV::BI##ID:
+#include "clang/Basic/BuiltinsRISCV.def"
+#undef BUILTIN
----------------
@vsk @HsiangKai I refactored this in rGd2f4b7d77828 to avoid the switch because it really messed up code coverage. It lead to a single .lcovfun section of a few gigabytes. That might be worth a look, and I recommend against this code pattern in the future.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100611



More information about the cfe-commits mailing list