----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2729/#review6042 -----------------------------------------------------------
Actually the Linux man page also says sync() is in unistd.h, so we must just be getting lucky there. We should just add the include unconditionally (i.e., get rid of the ifdef). - Steve Reinhardt On April 9, 2015, 2:29 a.m., Ruslan Bukin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2729/ > ----------------------------------------------------------- > > (Updated April 9, 2015, 2:29 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > include unistd.h for sync() > > > Diffs > ----- > > src/base/output.cc 235ff1c046df > > Diff: http://reviews.gem5.org/r/2729/diff/ > > > Testing > ------- > > tested on FreeBSD > > > Thanks, > > Ruslan Bukin > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
