Formatting and comments

Author: usama54321

lib/IRGen/IRGenSIL.cpp

@@ -965,9 +965,11 @@ class IRGenSILFunction :
   bool shouldShadowVariable(SILDebugVariable varInfo, bool isAnonymous) {
     return !IGM.IRGen.Opts.DisableDebuggerShadowCopies
       && !IGM.IRGen.Opts.shouldOptimize()
-      //Disable shadow in deserialized functions as they might be optimized
+      // Shadow copies are only emitted at -Onone, but a deserialized function
+      // might have been already optimized, so ignore those.
       && !CurSILFn->wasDeserializedCanonical()
-      && (!CurSILFn->isSpecialization() || !CurSILFn->getSpecializationInfo()->getParent()->wasDeserializedCanonical())
+      && (!CurSILFn->isSpecialization() ||
+          !CurSILFn->getSpecializationInfo()->getParent()->wasDeserializedCanonical())
       && !isAnonymous;
   }
 

lib/Serialization/SILFormat.h

@@ -316,12 +316,15 @@ namespace sil_block {
 
   using SILDebugValueLayout = BCRecordLayout<
     SIL_DEBUG_VALUE,
-    
-    SILTypeCategoryField, // operand type category
-    SILTypeCategoryField, // debug var type category
-    BCFixed<11>,// poison, movableValueDebuginfo, trace, hasDebugVar, isLet, isDenseMapSingleton(two bits), hasSource, hasLoc, hasExpr
-    BCArray<ValueIDField> // operand info: operand, type, debug var info: name, argno, optional stuff: typeid
-    >;
+
+    SILTypeCategoryField, /// operand type category
+    SILTypeCategoryField, /// debug var type category
+    BCFixed<11>,          /// poison, movableValueDebuginfo, trace,
+                          /// hasDebugVar, isLet, isDenseMapSingleton(two
+                          /// bits), hasSource, hasLoc, hasExpr
+    BCArray<ValueIDField> /// operand info: operand, type, debug var info:
+                          /// name, argno, optional stuff: typeid
+  >;
 
   using DebugValueDelimiterLayout = BCRecordLayout<
     SIL_DEBUG_VALUE_DELIMITER