[PATCH] D53891: [LTO] Pass down LTOUnit codegen flag to bitcode writer

2018-11-20 Thread Teresa Johnson via Phabricator via cfe-commits
tejohnson updated this revision to Diff 174835. tejohnson added a comment. Update to use new module flag Repository: rC Clang https://reviews.llvm.org/D53891 Files: include/clang/Driver/Options.td include/clang/Driver/SanitizerArgs.h include/clang/Frontend/CodeGenOptions.def

[PATCH] D53891: [LTO] Pass down LTOUnit codegen flag to bitcode writer

2018-11-16 Thread Teresa Johnson via Phabricator via cfe-commits
tejohnson added a comment. In https://reviews.llvm.org/D53891#1300789, @tejohnson wrote: > - Switch the default of splitting lto units to off by default, unless > compiled with CFI or -fwhole-program-vtables. Thinking ahead to when I add the index based WPD implementation, we are going to

[PATCH] D53891: [LTO] Pass down LTOUnit codegen flag to bitcode writer

2018-11-15 Thread Teresa Johnson via Phabricator via cfe-commits
tejohnson updated this revision to Diff 174332. tejohnson added a comment. Update a couple tests due to new default of splitting off and new flag. Repository: rC Clang https://reviews.llvm.org/D53891 Files: include/clang/Driver/Options.td include/clang/Driver/SanitizerArgs.h

[PATCH] D53891: [LTO] Pass down LTOUnit codegen flag to bitcode writer

2018-11-15 Thread Teresa Johnson via Phabricator via cfe-commits
tejohnson updated this revision to Diff 174329. tejohnson added a comment. - As discussed off-patch, will use a new, separate option to control this splitting, here I am using -f[no]split-lto-unit. - Switch the default of splitting lto units to off by default, unless compiled with CFI or

[PATCH] D53891: [LTO] Pass down LTOUnit codegen flag to bitcode writer

2018-10-30 Thread Teresa Johnson via Phabricator via cfe-commits
tejohnson created this revision. tejohnson added a reviewer: pcc. Herald added subscribers: cfe-commits, dexonsmith, steven_wu, eraman, inglorion, mehdi_amini. Clang side patch for recording the LTOUnit flag in the index. Depends on https://reviews.llvm.org/D53890. Repository: rC Clang