[PATCH] D67455: [Clang][CodeGen] support alias attribute w/ gnu_inline

Andrew Kelley via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 11 11:52:15 PDT 2019


andrewrk requested changes to this revision.
andrewrk added inline comments.
This revision now requires changes to proceed.


================
Comment at: clang/lib/AST/Decl.cpp:3352
+  assert((doesThisDeclarationHaveABody() || willHaveBody()) ||
+         hasAttr<AliasAttr>() && "Must be a function definition");
   assert(isInlined() && "Function must be inline");
----------------
It looks like the rparen is in the wrong place. I'd expect:

    assert((doesThisDeclarationHaveABody() || willHaveBody() ||
         hasAttr<AliasAttr>()) && "Must be a function definition");


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67455





More information about the cfe-commits mailing list