1#!/usr/bin/env python2.7 2 3"""Check CFC - Check Compile Flow Consistency 4 5This is a compiler wrapper for testing that code generation is consistent with 6different compilation processes. It checks that code is not unduly affected by 7compiler options or other changes which should not have side effects. 8 9To use: 10-Ensure that the compiler under test (i.e. clang, clang++) is on the PATH 11-On Linux copy this script to the name of the compiler 12 e.g. cp check_cfc.py clang && cp check_cfc.py clang++ 13-On Windows use setup.py to generate check_cfc.exe and copy that to clang.exe 14 and clang++.exe 15-Enable the desired checks in check_cfc.cfg (in the same directory as the 16 wrapper) 17 e.g. 18[Checks] 19dash_g_no_change = true 20dash_s_no_change = false 21 22-The wrapper can be run using its absolute path or added to PATH before the 23 compiler under test 24 e.g. export PATH=<path to check_cfc>:$PATH 25-Compile as normal. The wrapper intercepts normal -c compiles and will return 26 non-zero if the check fails. 27 e.g. 28$ clang -c test.cpp 29Code difference detected with -g 30--- /tmp/tmp5nv893.o 31+++ /tmp/tmp6Vwjnc.o 32@@ -1 +1 @@ 33- 0: 48 8b 05 51 0b 20 00 mov 0x200b51(%rip),%rax 34+ 0: 48 39 3d 51 0b 20 00 cmp %rdi,0x200b51(%rip) 35 36-To run LNT with Check CFC specify the absolute path to the wrapper to the --cc 37 and --cxx options 38 e.g. 39 lnt runtest nt --cc <path to check_cfc>/clang \\ 40 --cxx <path to check_cfc>/clang++ ... 41 42To add a new check: 43-Create a new subclass of WrapperCheck 44-Implement the perform_check() method. This should perform the alternate compile 45 and do the comparison. 46-Add the new check to check_cfc.cfg. The check has the same name as the 47 subclass. 48""" 49 50from __future__ import print_function 51 52import imp 53import os 54import platform 55import shutil 56import subprocess 57import sys 58import tempfile 59import ConfigParser 60import io 61 62import obj_diff 63 64def is_windows(): 65 """Returns True if running on Windows.""" 66 return platform.system() == 'Windows' 67 68class WrapperStepException(Exception): 69 """Exception type to be used when a step other than the original compile 70 fails.""" 71 def __init__(self, msg, stdout, stderr): 72 self.msg = msg 73 self.stdout = stdout 74 self.stderr = stderr 75 76class WrapperCheckException(Exception): 77 """Exception type to be used when a comparison check fails.""" 78 def __init__(self, msg): 79 self.msg = msg 80 81def main_is_frozen(): 82 """Returns True when running as a py2exe executable.""" 83 return (hasattr(sys, "frozen") or # new py2exe 84 hasattr(sys, "importers") or # old py2exe 85 imp.is_frozen("__main__")) # tools/freeze 86 87def get_main_dir(): 88 """Get the directory that the script or executable is located in.""" 89 if main_is_frozen(): 90 return os.path.dirname(sys.executable) 91 return os.path.dirname(sys.argv[0]) 92 93def remove_dir_from_path(path_var, directory): 94 """Remove the specified directory from path_var, a string representing 95 PATH""" 96 pathlist = path_var.split(os.pathsep) 97 norm_directory = os.path.normpath(os.path.normcase(directory)) 98 pathlist = filter(lambda x: os.path.normpath( 99 os.path.normcase(x)) != norm_directory, pathlist) 100 return os.pathsep.join(pathlist) 101 102def path_without_wrapper(): 103 """Returns the PATH variable modified to remove the path to this program.""" 104 scriptdir = get_main_dir() 105 path = os.environ['PATH'] 106 return remove_dir_from_path(path, scriptdir) 107 108def flip_dash_g(args): 109 """Search for -g in args. If it exists then return args without. If not then 110 add it.""" 111 if '-g' in args: 112 # Return args without any -g 113 return [x for x in args if x != '-g'] 114 else: 115 # No -g, add one 116 return args + ['-g'] 117 118def derive_output_file(args): 119 """Derive output file from the input file (if just one) or None 120 otherwise.""" 121 infile = get_input_file(args) 122 if infile is None: 123 return None 124 else: 125 return '{}.o'.format(os.path.splitext(infile)[0]) 126 127def get_output_file(args): 128 """Return the output file specified by this command or None if not 129 specified.""" 130 grabnext = False 131 for arg in args: 132 if grabnext: 133 return arg 134 if arg == '-o': 135 # Specified as a separate arg 136 grabnext = True 137 elif arg.startswith('-o'): 138 # Specified conjoined with -o 139 return arg[2:] 140 assert grabnext == False 141 142 return None 143 144def is_output_specified(args): 145 """Return true is output file is specified in args.""" 146 return get_output_file(args) is not None 147 148def replace_output_file(args, new_name): 149 """Replaces the specified name of an output file with the specified name. 150 Assumes that the output file name is specified in the command line args.""" 151 replaceidx = None 152 attached = False 153 for idx, val in enumerate(args): 154 if val == '-o': 155 replaceidx = idx + 1 156 attached = False 157 elif val.startswith('-o'): 158 replaceidx = idx 159 attached = True 160 161 if replaceidx is None: 162 raise Exception 163 replacement = new_name 164 if attached == True: 165 replacement = '-o' + new_name 166 args[replaceidx] = replacement 167 return args 168 169def add_output_file(args, output_file): 170 """Append an output file to args, presuming not already specified.""" 171 return args + ['-o', output_file] 172 173def set_output_file(args, output_file): 174 """Set the output file within the arguments. Appends or replaces as 175 appropriate.""" 176 if is_output_specified(args): 177 args = replace_output_file(args, output_file) 178 else: 179 args = add_output_file(args, output_file) 180 return args 181 182gSrcFileSuffixes = ('.c', '.cpp', '.cxx', '.c++', '.cp', '.cc') 183 184def get_input_file(args): 185 """Return the input file string if it can be found (and there is only 186 one).""" 187 inputFiles = list() 188 for arg in args: 189 testarg = arg 190 quotes = ('"', "'") 191 while testarg.endswith(quotes): 192 testarg = testarg[:-1] 193 testarg = os.path.normcase(testarg) 194 195 # Test if it is a source file 196 if testarg.endswith(gSrcFileSuffixes): 197 inputFiles.append(arg) 198 if len(inputFiles) == 1: 199 return inputFiles[0] 200 else: 201 return None 202 203def set_input_file(args, input_file): 204 """Replaces the input file with that specified.""" 205 infile = get_input_file(args) 206 if infile: 207 infile_idx = args.index(infile) 208 args[infile_idx] = input_file 209 return args 210 else: 211 # Could not find input file 212 assert False 213 214def is_normal_compile(args): 215 """Check if this is a normal compile which will output an object file rather 216 than a preprocess or link. args is a list of command line arguments.""" 217 compile_step = '-c' in args 218 # Bitcode cannot be disassembled in the same way 219 bitcode = '-flto' in args or '-emit-llvm' in args 220 # Version and help are queries of the compiler and override -c if specified 221 query = '--version' in args or '--help' in args 222 # Options to output dependency files for make 223 dependency = '-M' in args or '-MM' in args 224 # Check if the input is recognised as a source file (this may be too 225 # strong a restriction) 226 input_is_valid = bool(get_input_file(args)) 227 return compile_step and not bitcode and not query and not dependency and input_is_valid 228 229def run_step(command, my_env, error_on_failure): 230 """Runs a step of the compilation. Reports failure as exception.""" 231 # Need to use shell=True on Windows as Popen won't use PATH otherwise. 232 p = subprocess.Popen(command, stdout=subprocess.PIPE, 233 stderr=subprocess.PIPE, env=my_env, shell=is_windows()) 234 (stdout, stderr) = p.communicate() 235 if p.returncode != 0: 236 raise WrapperStepException(error_on_failure, stdout, stderr) 237 238def get_temp_file_name(suffix): 239 """Get a temporary file name with a particular suffix. Let the caller be 240 reponsible for deleting it.""" 241 tf = tempfile.NamedTemporaryFile(suffix=suffix, delete=False) 242 tf.close() 243 return tf.name 244 245class WrapperCheck(object): 246 """Base class for a check. Subclass this to add a check.""" 247 def __init__(self, output_file_a): 248 """Record the base output file that will be compared against.""" 249 self._output_file_a = output_file_a 250 251 def perform_check(self, arguments, my_env): 252 """Override this to perform the modified compilation and required 253 checks.""" 254 raise NotImplementedError("Please Implement this method") 255 256class dash_g_no_change(WrapperCheck): 257 def perform_check(self, arguments, my_env): 258 """Check if different code is generated with/without the -g flag.""" 259 output_file_b = get_temp_file_name('.o') 260 261 alternate_command = list(arguments) 262 alternate_command = flip_dash_g(alternate_command) 263 alternate_command = set_output_file(alternate_command, output_file_b) 264 run_step(alternate_command, my_env, "Error compiling with -g") 265 266 # Compare disassembly (returns first diff if differs) 267 difference = obj_diff.compare_object_files(self._output_file_a, 268 output_file_b) 269 if difference: 270 raise WrapperCheckException( 271 "Code difference detected with -g\n{}".format(difference)) 272 273 # Clean up temp file if comparison okay 274 os.remove(output_file_b) 275 276class dash_s_no_change(WrapperCheck): 277 def perform_check(self, arguments, my_env): 278 """Check if compiling to asm then assembling in separate steps results 279 in different code than compiling to object directly.""" 280 output_file_b = get_temp_file_name('.o') 281 282 alternate_command = arguments + ['-via-file-asm'] 283 alternate_command = set_output_file(alternate_command, output_file_b) 284 run_step(alternate_command, my_env, 285 "Error compiling with -via-file-asm") 286 287 # Compare if object files are exactly the same 288 exactly_equal = obj_diff.compare_exact(self._output_file_a, output_file_b) 289 if not exactly_equal: 290 # Compare disassembly (returns first diff if differs) 291 difference = obj_diff.compare_object_files(self._output_file_a, 292 output_file_b) 293 if difference: 294 raise WrapperCheckException( 295 "Code difference detected with -S\n{}".format(difference)) 296 297 # Code is identical, compare debug info 298 dbgdifference = obj_diff.compare_debug_info(self._output_file_a, 299 output_file_b) 300 if dbgdifference: 301 raise WrapperCheckException( 302 "Debug info difference detected with -S\n{}".format(dbgdifference)) 303 304 raise WrapperCheckException("Object files not identical with -S\n") 305 306 # Clean up temp file if comparison okay 307 os.remove(output_file_b) 308 309if __name__ == '__main__': 310 # Create configuration defaults from list of checks 311 default_config = """ 312[Checks] 313""" 314 315 # Find all subclasses of WrapperCheck 316 checks = [cls.__name__ for cls in vars()['WrapperCheck'].__subclasses__()] 317 318 for c in checks: 319 default_config += "{} = false\n".format(c) 320 321 config = ConfigParser.RawConfigParser() 322 config.readfp(io.BytesIO(default_config)) 323 scriptdir = get_main_dir() 324 config_path = os.path.join(scriptdir, 'check_cfc.cfg') 325 try: 326 config.read(os.path.join(config_path)) 327 except: 328 print("Could not read config from {}, " 329 "using defaults.".format(config_path)) 330 331 my_env = os.environ.copy() 332 my_env['PATH'] = path_without_wrapper() 333 334 arguments_a = list(sys.argv) 335 336 # Prevent infinite loop if called with absolute path. 337 arguments_a[0] = os.path.basename(arguments_a[0]) 338 339 # Sanity check 340 enabled_checks = [check_name 341 for check_name in checks 342 if config.getboolean('Checks', check_name)] 343 checks_comma_separated = ', '.join(enabled_checks) 344 print("Check CFC, checking: {}".format(checks_comma_separated)) 345 346 # A - original compilation 347 output_file_orig = get_output_file(arguments_a) 348 if output_file_orig is None: 349 output_file_orig = derive_output_file(arguments_a) 350 351 p = subprocess.Popen(arguments_a, env=my_env, shell=is_windows()) 352 p.communicate() 353 if p.returncode != 0: 354 sys.exit(p.returncode) 355 356 if not is_normal_compile(arguments_a) or output_file_orig is None: 357 # Bail out here if we can't apply checks in this case. 358 # Does not indicate an error. 359 # Maybe not straight compilation (e.g. -S or --version or -flto) 360 # or maybe > 1 input files. 361 sys.exit(0) 362 363 # Sometimes we generate files which have very long names which can't be 364 # read/disassembled. This will exit early if we can't find the file we 365 # expected to be output. 366 if not os.path.isfile(output_file_orig): 367 sys.exit(0) 368 369 # Copy output file to a temp file 370 temp_output_file_orig = get_temp_file_name('.o') 371 shutil.copyfile(output_file_orig, temp_output_file_orig) 372 373 # Run checks, if they are enabled in config and if they are appropriate for 374 # this command line. 375 current_module = sys.modules[__name__] 376 for check_name in checks: 377 if config.getboolean('Checks', check_name): 378 class_ = getattr(current_module, check_name) 379 checker = class_(temp_output_file_orig) 380 try: 381 checker.perform_check(arguments_a, my_env) 382 except WrapperCheckException as e: 383 # Check failure 384 print("{} {}".format(get_input_file(arguments_a), e.msg), file=sys.stderr) 385 386 # Remove file to comply with build system expectations (no 387 # output file if failed) 388 os.remove(output_file_orig) 389 sys.exit(1) 390 391 except WrapperStepException as e: 392 # Compile step failure 393 print(e.msg, file=sys.stderr) 394 print("*** stdout ***", file=sys.stderr) 395 print(e.stdout, file=sys.stderr) 396 print("*** stderr ***", file=sys.stderr) 397 print(e.stderr, file=sys.stderr) 398 399 # Remove file to comply with build system expectations (no 400 # output file if failed) 401 os.remove(output_file_orig) 402 sys.exit(1) 403