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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits