angroyer created this revision.
angroyer added a reviewer: LLVM.
angroyer added a project: LLVM.
Herald added a reviewer: NoQ.
Herald added a project: All.
angroyer requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

The CmdArgsSansAnalyses array does not contain the "-target" argument required 
to define a valid "-triple" argument in the html info files. This patch aims to 
fix that.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D138577

Files:
  clang/tools/scan-build/libexec/ccc-analyzer


Index: clang/tools/scan-build/libexec/ccc-analyzer
===================================================================
--- clang/tools/scan-build/libexec/ccc-analyzer
+++ clang/tools/scan-build/libexec/ccc-analyzer
@@ -226,6 +226,10 @@
   else {
     $Cmd = $Clang;
 
+    if (defined $AnalyzerTarget) {
+      push @Args, "-target", $AnalyzerTarget;
+    }
+
     # Create arguments for doing regular parsing.
     my $SyntaxArgs = GetCCArgs($HtmlDir, "-fsyntax-only", \@Args);
     @CmdArgsSansAnalyses = @$SyntaxArgs;
@@ -245,10 +249,6 @@
       push @Args, "-Xclang", $arg;
     }
 
-    if (defined $AnalyzerTarget) {
-      push @Args, "-target", $AnalyzerTarget;
-    }
-
     my $AnalysisArgs = GetCCArgs($HtmlDir, "--analyze", \@Args);
     @CmdArgs = @$AnalysisArgs;
   }


Index: clang/tools/scan-build/libexec/ccc-analyzer
===================================================================
--- clang/tools/scan-build/libexec/ccc-analyzer
+++ clang/tools/scan-build/libexec/ccc-analyzer
@@ -226,6 +226,10 @@
   else {
     $Cmd = $Clang;
 
+    if (defined $AnalyzerTarget) {
+      push @Args, "-target", $AnalyzerTarget;
+    }
+
     # Create arguments for doing regular parsing.
     my $SyntaxArgs = GetCCArgs($HtmlDir, "-fsyntax-only", \@Args);
     @CmdArgsSansAnalyses = @$SyntaxArgs;
@@ -245,10 +249,6 @@
       push @Args, "-Xclang", $arg;
     }
 
-    if (defined $AnalyzerTarget) {
-      push @Args, "-target", $AnalyzerTarget;
-    }
-
     my $AnalysisArgs = GetCCArgs($HtmlDir, "--analyze", \@Args);
     @CmdArgs = @$AnalysisArgs;
   }
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D138577: Use proper... Anthony Groyer via Phabricator via cfe-commits

Reply via email to