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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to