[PATCH] Add control flow integrity diagnosis function to UBSan	runtime library.
    Alexey Samsonov 
    vonosmas at gmail.com
       
    Thu Jun 18 17:51:58 PDT 2015
    
    
  
LGTM
================
Comment at: test/cfi/lit.cfg:12
@@ -11,2 +11,3 @@
 if config.lto_supported:
-  config.substitutions.append((r"%clangxx_cfi ", ' '.join(config.lto_launch + [clangxx] + config.lto_flags + ['-fsanitize=cfi '])))
+  clangxx_cfi = ' '.join(config.lto_launch + [clangxx] + config.lto_flags + ['-fsanitize=cfi '])
+  config.substitutions.append((r"%clangxx_cfi ", clangxx_cfi))
----------------
I think it would be easier to construct list of args for clangxx_cfi, and clangxx_cfi_diag, and then do `' '.join(...)` for both of them. Leaving this up to you, though.
http://reviews.llvm.org/D10269
EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
    
    
More information about the llvm-commits
mailing list