Split out BitmapProcState opts into separate cpp files
This CL includes two new things:
1) Slicing SkOpts by functionality, in addition to architecture.
2) Enabling architectures within the translation unit, rather than at
the command line.
The first attempt at #2 failed, because of a clang bug. However, that
bug only impacts code that passes AVX vectors to functions. That pattern
only happens in RasterPipeline, so all other SkOpts can be peeled off
this way.
I think it's possible to go further, and fold all the architecture
specializations of a particular module (eg BitmapProcState) into a
single cpp file. I started down that path, but realized the skvx
includes would get pretty tricky. So I backed up and got this version
working first. This is still an improvement:
- Clients don't need to know about special compile flags for any of the
newly added "opts" cpp files. They just get added to the soruce lists
like everything else.
- The initialization happens automatically, because I found a good spot
to trigger it when we're choosing procs.
Bug: skia:13983
Bug: skia:14355
Change-Id: Ibdcff5fbef9df25416cfd90eb1680153e54e7661
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/727257
Reviewed-by: Kevin Lubick <kjlubick@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
15 files changed