Address more comments

Author: vgvassilev

clang/include/clang/Interpreter/Interpreter.h

@@ -95,8 +95,7 @@ class Interpreter {
   // An optional parser for CUDA offloading
   std::unique_ptr<IncrementalParser> DeviceParser;
 
-  /// List containing every information about every incrementally parsed piece
-  /// of code.
+  /// List containing information about each incrementally parsed piece of code.
   std::list<PartialTranslationUnit> PTUs;
 
   unsigned InitPTUSize = 0;

clang/lib/Interpreter/DeviceOffload.h

@@ -49,7 +49,7 @@ class IncrementalCUDADeviceParser : public IncrementalParser {
   llvm::SmallString<1024> PTXCode;
   llvm::SmallVector<char, 1024> FatbinContent;
   llvm::IntrusiveRefCntPtr<llvm::vfs::InMemoryFileSystem> VFS;
-  CodeGenOptions &CodeGenOpts; // intentionally a reference.
+  CodeGenOptions &CodeGenOpts; // Intentionally a reference.
   const TargetOptions &TargetOpts;
 };
 

clang/lib/Interpreter/Interpreter.cpp

@@ -302,22 +302,15 @@ class IncrementalAction : public WrapperFrontendAction {
                 CI.getFrontendOpts().ProgramAction);
             return Act;
           case frontend::ASTDump:
-            [[fallthrough]];
           case frontend::ASTPrint:
-            [[fallthrough]];
           case frontend::ParseSyntaxOnly:
             Act = CreateFrontendAction(CI);
             break;
           case frontend::PluginAction:
-            [[fallthrough]];
           case frontend::EmitAssembly:
-            [[fallthrough]];
           case frontend::EmitBC:
-            [[fallthrough]];
           case frontend::EmitObj:
-            [[fallthrough]];
           case frontend::PrintPreprocessedInput:
-            [[fallthrough]];
           case frontend::EmitLLVMOnly:
             Act.reset(new EmitLLVMOnlyAction(&LLVMCtx));
             break;