----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3745/#review9172 -----------------------------------------------------------
Ship it! Looks ok, but the formatting of the patch summary is way outside the 65 char limit. - Andreas Hansson On Nov. 23, 2016, 10:25 p.m., Joe Gross wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3745/ > ----------------------------------------------------------- > > (Updated Nov. 23, 2016, 10:25 p.m.) > > > Review request for Default and Tony Gutierrez. > > > Repository: gem5 > > > Description > ------- > > scons: fix sanitizer flags with multiple sanitizers > > There has been some problem when using address and undefined-behavior > sanitizers at the same time. > This patch will look for the special case where both are enabled at once and > change the flags passed > to the compiler to reflect this. > > > Diffs > ----- > > src/SConscript 9e5050028323260c6d2211d7158ac4a4145ed8bc > > Diff: http://reviews.gem5.org/r/3745/diff/ > > > Testing > ------- > > local build > > > Thanks, > > Joe Gross > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
