[clang-format] Don't remove parentheses in macro definitions (#81444)

Closes #81399.

Author: owenca

clang/lib/Format/UnwrappedLineParser.cpp

@@ -2518,7 +2518,7 @@ bool UnwrappedLineParser::parseParens(TokenType AmpAmpTokenType) {
  parseChildBlock();
  break;
  case tok::r_paren:
- if (!MightBeStmtExpr &&
+ if (!MightBeStmtExpr && !Line->InMacroBody &&
  Style.RemoveParentheses > FormatStyle::RPS_Leave) {
  const auto *Prev = LeftParen->Previous;
  const auto *Next = Tokens->peekNextToken();

clang/unittests/Format/FormatTest.cpp

@@ -26972,6 +26972,7 @@ TEST_F(FormatTest, RemoveParentheses) {
  EXPECT_EQ(Style.RemoveParentheses, FormatStyle::RPS_Leave);
 
  Style.RemoveParentheses = FormatStyle::RPS_MultipleParentheses;
+ verifyFormat("#define Foo(...) foo((__VA_ARGS__))", Style);
  verifyFormat("int x __attribute__((aligned(16))) = 0;", Style);
  verifyFormat("decltype((foo->bar)) baz;", Style);
  verifyFormat("class __declspec(dllimport) X {};",
@@ -27006,6 +27007,7 @@ TEST_F(FormatTest, RemoveParentheses) {
  verifyFormat("return (({ 0; }));", "return ((({ 0; })));", Style);
 
  Style.RemoveParentheses = FormatStyle::RPS_ReturnStatement;
+ verifyFormat("#define Return0 return (0);", Style);
  verifyFormat("return 0;", "return (0);", Style);
  verifyFormat("co_return 0;", "co_return ((0));", Style);
  verifyFormat("return 0;", "return (((0)));", Style);