[PATCH] D94741: [Utils][WIP] Check for more global information in update_test_checks
    Mircea Trofin via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Feb  8 07:04:38 PST 2021
    
    
  
mtrofin added inline comments.
Herald added a reviewer: bollu.
================
Comment at: llvm/utils/UpdateTestChecks/common.py:377
 class NamelessValue:
-    def __init__(self, check_prefix, ir_prefix, ir_regexp):
+    def __init__(self, check_prefix, check_key, ir_prefix, global_ir_prefix, global_ir_prefix_regexp, ir_regexp, global_ir_rhs_regexp, is_before_functions):
         self.check_prefix = check_prefix
----------------
nit: could you run a linter to split, insomuch as possible, long lines? I (think I) know we don't enforce a code style of .py, but I'd argue there's readability value in doing it. (no need to scroll right, less eye movement effort)
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94741/new/
https://reviews.llvm.org/D94741
    
    
More information about the llvm-commits
mailing list