api-digester: in module dump, record whether a func decl is mutating.

Author: nkcsgexi

test/api-digester/Inputs/cake.swift

@@ -1,4 +1,4 @@
 public struct S1 {
   public func foo1() {}
-  public func foo2() {}
+  mutating public func foo2() {}
 }

test/api-digester/Outputs/cake.json

@@ -36,6 +36,7 @@
           "usr": "s:FV4cake2S14foo2FT_T_",
           "location": "",
           "moduleName": "cake",
+          "mutating": true,
           "children": [
             {
               "kind": "TypeNominal",

tools/swift-api-digester/DigesterEnums.def

@@ -66,6 +66,7 @@ KEY(children)
 KEY(printedName)
 KEY(moduleName)
 KEY(throwing)
+KEY(mutating)
 KEY(typeAttributes)
 KEY(declAttributes)
 KEY(declKind)

tools/swift-api-digester/swift-api-digester.cpp

@@ -261,6 +261,7 @@ struct SDKNodeInitInfo {
   StringRef Location;
   StringRef ModuleName;
   bool IsThrowing = false;
+  bool IsMutating = false;
   Optional<uint8_t> SelfIndex;
   std::vector<SDKDeclAttrKind> DeclAttrs;
   std::vector<TypeAttrKind> TypeAttrs;
@@ -683,15 +684,18 @@ class SDKNodeVar : public SDKNodeDecl {
 
 class SDKNodeAbstractFunc : public SDKNodeDecl {
   const bool IsThrowing;
+  const bool IsMutating;
   const Optional<uint8_t> SelfIndex;
 
 protected:
   SDKNodeAbstractFunc(SDKNodeInitInfo Info, SDKNodeKind Kind) :
                                                     SDKNodeDecl(Info, Kind),
                                                     IsThrowing(Info.IsThrowing),
+                                                    IsMutating(Info.IsMutating),
                                                     SelfIndex(Info.SelfIndex){}
 public:
   bool isThrowing() const { return IsThrowing; }
+  bool isMutating() const { return IsMutating; }
   uint8_t getSelfIndex() const { return SelfIndex.getValue(); }
   Optional<uint8_t> getSelfIndexOptional() const { return SelfIndex; }
   bool hasSelfIndex() const { return SelfIndex.hasValue(); }
@@ -800,6 +804,8 @@ NodeUniquePtr SDKNode::constructSDKNode(llvm::yaml::MappingNode *Node) {
       Info.ModuleName = GetScalarString(Pair.getValue());
     } else if (Key == Key_throwing) {
       Info.IsThrowing = true;
+    } else if (Key == Key_mutating) {
+      Info.IsMutating = true;
     } else if (Key == Key_typeAttributes) {
       auto *Seq = cast<llvm::yaml::SequenceNode>(Pair.getValue());
       for (auto It = Seq->begin(); It != Seq->end(); ++ It) {
@@ -946,6 +952,13 @@ static bool isFuncThrowing(ValueDecl *VD) {
   return false;
 }
 
+static bool isFuncMutating(ValueDecl *VD) {
+  if (auto AF = dyn_cast<FuncDecl>(VD)) {
+    return AF->isMutating();
+  }
+  return false;
+}
+
 static Optional<uint8_t> getSelfIndex(ValueDecl *VD) {
   if (auto AF = dyn_cast<AbstractFunctionDecl>(VD)) {
     if (AF->isImportAsInstanceMember())
@@ -965,7 +978,8 @@ SDKNodeInitInfo::SDKNodeInitInfo(ValueDecl *VD) :
     PrintedName(getPrintedName(VD)), DKind(VD->getKind()),
     USR(calculateUsr(VD)), Location(calculateLocation(VD)),
     ModuleName(VD->getModuleContext()->getName().str()),
-    IsThrowing(isFuncThrowing(VD)), SelfIndex(getSelfIndex(VD)) {
+    IsThrowing(isFuncThrowing(VD)), IsMutating(isFuncMutating(VD)),
+    SelfIndex(getSelfIndex(VD)) {
   if (VD->getAttrs().getDeprecated(VD->getASTContext()))
     DeclAttrs.push_back(SDKDeclAttrKind::DAK_deprecated);
 }
@@ -1296,6 +1310,8 @@ namespace swift {
           if (auto F = dyn_cast<SDKNodeAbstractFunc>(value.get())) {
             if (bool isThrowing = F->isThrowing())
               out.mapRequired(Key_throwing, isThrowing);
+            if (bool isMutating = F->isMutating())
+              out.mapRequired(Key_mutating, isMutating);
             if (F->hasSelfIndex()) {
               auto Index = F->getSelfIndex();
               out.mapRequired(Key_selfIndex, Index);