[PATCH] D121283: [Clang] Support multiple attributes in a single pragma
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Mar 15 09:58:59 PDT 2022
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
Aside from a minor testing nit, this LGTM. Thank you for the new functionality!
================
Comment at: clang/test/Parser/pragma-multiple-attributes.cpp:1-11
+// RUN: %clang_cc1 -Wno-pragma-clang-attribute -verify %s
+
+#pragma clang attribute push ([[clang::disable_tail_calls, noreturn]], apply_to = function)
+#pragma clang attribute pop
+
+#pragma clang attribute push (__attribute__((disable_tail_calls, annotate("test"))), apply_to = function)
+#pragma clang attribute pop
----------------
I think this entire test file can now be folded back into `pragma-attribute.cpp` (this way we don't have to pay the overhead of another execution of Clang to test the functionality).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121283/new/
https://reviews.llvm.org/D121283
More information about the cfe-commits
mailing list