[Mesa-dev] [PATCH v4] swr: Refactor checks for compiler feature flags

Chuck Atkins chuck.atkins at kitware.com
Fri Jul 1 15:03:58 UTC 2016


>
> This part should have been a separate patch. Please try to keep things
> separate for the future.
>

Indeed, I should have this as two separate commits, one to encapsulate the
flag test and another to add additional options to test for.  I'll keep
them more segmented in the future.



> Esp with the IFS override this looks rather nasty imho.
>

I hate messing with IFS as it tends to get ugly real fast, I just couldn't
think of a better way at the time.  After sitting with it for a bit longer
now, I've updated the swr_require_cxx_feature_flags function now to not use
IFS.  Would you rather I push it as a new patch or just leave it alone for
now?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160701/4fcb34ab/attachment.html>


More information about the mesa-dev mailing list