| 2002 2003 2004 2005 2006 2007 2008 2009 2010 2011 2012 2013 2014 2015 2016 <2017> 2018 2019 2020 2021 2022 2023 2024 | Index | 2002 2003 2004 2005 2006 2007 2008 2009 2010 2011 2012 2013 2014 2015 2016 <2017> 2018 2019 2020 2021 2022 2023 2024 |
| <== Date ==> | <== Thread ==> |
|---|
| Subject: | Re: EPICS 7.0.1-pre1 (epicsException.h) |
| From: | Michael Davidsaver <[email protected]> |
| To: | Andrew Johnson <[email protected]>, [email protected] |
| Date: | Tue, 28 Nov 2017 14:46:02 -0600 |
On 11/28/2017 12:45 PM, Andrew Johnson wrote: >> I don't like removing FORCE_INLINE from castUnsafe(). This needs >> further analysis and a more target fix. >> >> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=54965 > Awaiting your suggestions there then. > My thought is to replace the jump table with nested switch(). So far this is just in my repo. I guess we would have to wait for Dirk to find out if this actually fixes the problem he sees. I had a look and this change does result in slightly shorter code, presumably because the gcc optimizer has an easier time dealing with switch(). So this change may be work including on its own merits. https://github.com/mdavidsaver/pvDataCPP/commit/156e7563dbcaf585e704ead631b0902fef12fdb2