[libcxx-commits] [PATCH] D84801: [libcxx] [test] XFAIL midpoint.float.pass on AArch64 with compiler-rt

[Sergej Jaskiewicz via Phabricator via libcxx-commits]

broadwaylamb added inline comments.


================
Comment at: libcxx/utils/libcxx/test/config.py:283
+
+        if self.get_lit_conf('use_compiler_rt'):
+            self.config.available_features.add('use_compiler_rt')
----------------
broadwaylamb wrote:
> ldionne wrote:
> > Not super happy about this for a couple of reasons:
> > 
> > 1. It adds more to `config.py`, which I'm trying to get rid of.
> > 2. There's nothing associated to the `use_compiler_rt` feature except the Lit feature itself. It's kind of like adding a new feature whose sole purpose is to `XFAIL` this specific test. Instead, it might make more sense to detect whether subnormal floats are supported?
> > 2. There's nothing associated to the `use_compiler_rt` feature except the Lit feature itself. It's kind of like adding a new feature whose sole purpose is to `XFAIL` this specific test. Instead, it might make more sense to detect whether subnormal floats are supported?
> 
> Do you mean like compile and run a test program that makes use of subnormal floats and generate a Lit feature based on whether it succeeds?
> 
> 
@ldionne could you please explain what you've meant?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84801