[PATCH] D77632: [TLI] Per-function fveclib for math library used for vectorization

[Mehdi AMINI via Phabricator via cfe-commits]

mehdi_amini added a comment.

In D77632#1976231 <https://reviews.llvm.org/D77632#1976231>, @wenlei wrote:

> And agree with @tejohnson, if the openness is a feature, it should be covered in tests, otherwise it can feel somewhat like a loophole and prone to breakage


The thing is that LLVM does not have much C++ unittests in general, so most of the "features" like this one that LLVM offers as a library are just an artifact of being "designed as a library" and being mindful about the layering.
>From this point of view this patch is changing the design of a component that was modular/pluggable into a closed system. I'm perfectly fine with trying to add a unit-test, I just don't know yet where it would fit in the LLVM testing though.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77632/new/

https://reviews.llvm.org/D77632