[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