----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25191/#review52506 -----------------------------------------------------------
Ship it! Ship It! 3rdparty/libprocess/3rdparty/stout/include/stout/unreachable.hpp <https://reviews.apache.org/r/25191/#comment91310> can we use __builtin_unreachable instead? we should probably check for it (__has_builtin(__builtin_unreachable)). - Dominic Hamon On Aug. 29, 2014, 10:53 a.m., Patrick Reilly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25191/ > ----------------------------------------------------------- > > (Updated Aug. 29, 2014, 10:53 a.m.) > > > Review request for mesos, Benjamin Hindman and Dominic Hamon. > > > Bugs: MESOS-1744 > https://issues.apache.org/jira/browse/MESOS-1744 > > > Repository: mesos-git > > > Description > ------- > > Use compiler intrinsics for unreachable, exit, and abort > Makes the functions not need to pretend to return something while > still silencing the compiler warnings. > > > Diffs > ----- > > 3rdparty/libprocess/3rdparty/stout/include/stout/abort.hpp f20feea > 3rdparty/libprocess/3rdparty/stout/include/stout/exit.hpp aaccbb4 > 3rdparty/libprocess/3rdparty/stout/include/stout/os.hpp 5bbf829 > 3rdparty/libprocess/3rdparty/stout/include/stout/os/sendfile.hpp 5607896 > 3rdparty/libprocess/3rdparty/stout/include/stout/unreachable.hpp 3568886 > > Diff: https://reviews.apache.org/r/25191/diff/ > > > Testing > ------- > > Make check runs. > > > Thanks, > > Patrick Reilly > >
