[PATCH] D130188: [DependenceAnalysis][PR56275] Normalize dependence analysis results to be non-negative when required
    Michael Kruse via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Aug  2 12:08:55 PDT 2022
    
    
  
Meinersbur accepted this revision.
Meinersbur added a comment.
This revision is now accepted and ready to land.
LGTM, but with a renaming suggestion.
================
Comment at: llvm/lib/Passes/PassBuilder.cpp:854
+Expected<bool> parseDependenceAnalysisPrinterOptions(StringRef Params) {
+  return parseSinglePassOption(Params, "da-print-normalized-results",
+                               "DependenceAnalysisPrinter");
----------------
I think the `da-print-` prefix is unnecessary since this is parsed for `print<da>` pass only.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130188/new/
https://reviews.llvm.org/D130188
    
    
More information about the llvm-commits
mailing list