[PATCH] D81227: Correctly report modified status for GCOVProfiling

serge via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 5 00:30:41 PDT 2020


serge-sans-paille created this revision.
serge-sans-paille added reviewers: calixte, jyknight, nikic, foad, jdoerfert.
Herald added subscribers: llvm-commits, hiraditya.
Herald added a project: LLVM.

Related to https://reviews.llvm.org/D80916


https://reviews.llvm.org/D81227

Files:
  llvm/lib/Transforms/Instrumentation/GCOVProfiling.cpp


Index: llvm/lib/Transforms/Instrumentation/GCOVProfiling.cpp
===================================================================
--- llvm/lib/Transforms/Instrumentation/GCOVProfiling.cpp
+++ llvm/lib/Transforms/Instrumentation/GCOVProfiling.cpp
@@ -568,8 +568,8 @@
   ExcludeRe = createRegexesFromString(Options.Exclude);
 
   if (Options.EmitNotes) emitProfileNotes();
-  if (Options.EmitData) return emitProfileArcs();
-  return false;
+  if (Options.EmitData) emitProfileArcs();
+  return true; // Because of the added flush.
 }
 
 PreservedAnalyses GCOVProfilerPass::run(Module &M,


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D81227.268678.patch
Type: text/x-patch
Size: 586 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200605/ae862a00/attachment.bin>


More information about the llvm-commits mailing list