Remove a misleading comment in CppCompileAction (the class is not actually overriding getInputs()).

--
MOS_MIGRATED_REVID=99173247

1 files changed, 1 insertions, 5 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileAction.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileAction.java
index 4839a36388..bd959611c7 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileAction.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileAction.java
@@ -225,10 +225,8 @@ public class CppCompileAction extends AbstractAction implements IncludeScannable
       UUID actionClassId,
       boolean usePic,
       RuleContext ruleContext) {
-    // getInputs() method is overridden in this class so we pass a dummy empty
-    // list to the AbstractAction constructor in place of a real input collection.
     super(owner,
-          Artifact.NO_ARTIFACTS,
+          createInputs(mandatoryInputs, context.getCompilationPrerequisites(), optionalSourceFile),
           CollectionUtils.asListWithoutNulls(outputFile, dotdFile.artifact(),
               gcnoFile, dwoFile));
     this.configuration = configuration;
@@ -254,8 +252,6 @@ public class CppCompileAction extends AbstractAction implements IncludeScannable
     // We do not need to include the middleman artifact since it is a generated
     // artifact and will definitely exist prior to this action execution.
     this.mandatoryInputs = mandatoryInputs;
-    setInputs(createInputs(mandatoryInputs, context.getCompilationPrerequisites(),
-        optionalSourceFile));
     verifyIncludePaths(ruleContext);
   }