Ok. For non-autofdo path, I think a warning (even an error) is needed in case of mismatch.
David On Mon, May 12, 2014 at 4:35 PM, Rong Xu <x...@google.com> wrote: > Hi, > > This patch adds an extra check of the incompatibility of use and saved > command line for the primary module. If the check files, we skip the > inclusion of all aux modules. > Maybe we should check the aux module command line with the > use-compilation command line directly. But I keep the old way that > checks b/w saved command lines. > > Tested with the program exposed this issue and internal benchmarks. > This check never failed for instrumentation based lipo in the test. > > Thanks, > > -Rong
