[PATCH] D82786: [llvm-ar] Unsupport test on FreeBSD

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 29 10:49:10 PDT 2020


MaskRay added a comment.

In D82786#2120469 <https://reviews.llvm.org/D82786#2120469>, @arichardson wrote:

> This depends on the underlying file system and FreeBSD version. However, I guess marking it as unsupported is the simplest solution to avoid test failures.
>
> > ! In D82786#2120498 <https://reviews.llvm.org/D82786#2120498>, @kevans wrote:
> > 
> >> ! In D82786#2120469 <https://reviews.llvm.org/D82786#2120469>, @arichardson wrote:
> > 
> > This depends on the underlying file system and FreeBSD version. However, I guess marking it as unsupported is the simplest solution to avoid test failures.
>
> I concur... the change in FreeBSD that stops allowing it by default for all filesystems will get MFC'd and make it into 12.2 if there's not significant need to flip it back on, but stable/11 will remain supported for some time yet.


Can you rewrite the comment to make it more accurate?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82786





More information about the llvm-commits mailing list