[llvm] r264310 - don't hardcode the name of the llc checks script

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 24 10:30:38 PDT 2016

Author: spatel
Date: Thu Mar 24 12:30:38 2016
New Revision: 264310

URL: http://llvm.org/viewvc/llvm-project?rev=264310&view=rev
don't hardcode the name of the llc checks script

We lose the 'utils' directory name in our advertising line with
this change. We could retain that, but I don't see the point.
This removes a dependency for making the script apply to more than
'llc'. Ie, we'll want to change the script name if it works with 
opt/clang too.


Modified: llvm/trunk/utils/update_llc_test_checks.py
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/update_llc_test_checks.py?rev=264310&r1=264309&r2=264310&view=diff
--- llvm/trunk/utils/update_llc_test_checks.py (original)
+++ llvm/trunk/utils/update_llc_test_checks.py Thu Mar 24 12:30:38 2016
@@ -9,8 +9,7 @@ a single test function.
 import argparse
 import itertools
-# Could be used to advertise this file's name ("autogenerated_note").
-# import os
+import os         # Used to advertise this file's name ("autogenerated_note").
 import string
 import subprocess
 import sys
@@ -144,10 +143,8 @@ def main():
   parser.add_argument('tests', nargs='+')
   args = parser.parse_args()
-  # FIXME: we don't need to hardcode this name.
   autogenerated_note = ('; NOTE: Assertions have been autogenerated by '
-                        'utils/update_llc_test_checks.py')
-                        # + os.path.basename(__file__))
+                        + os.path.basename(__file__))
   for test in args.tests:
     if args.verbose:

More information about the llvm-commits mailing list