[PATCH] D86879: [XCOFF][AIX] Handle TOC entries that could not be reached by positive range in small code model
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 16 07:30:44 PDT 2020
fhahn added a comment.
In D86879#2274099 <https://reviews.llvm.org/D86879#2274099>, @jasonliu wrote:
> In D86879#2274076 <https://reviews.llvm.org/D86879#2274076>, @nemanjai wrote:
>
>> In D86879#2274025 <https://reviews.llvm.org/D86879#2274025>, @hubert.reinterpretcast wrote:
>>
>>> We believe that the test is necessarily large. Has the associated bot encountered similar issues before? What was the recommended fix to retain the test in more appropriate environments?
>>
>> Perhaps the test case should be restricted to some key platforms where cross-compilation to AIX is relevant? Perhaps `Linux` (PPC is already required).
>
> I could add
> `# REQUIRES: aix || linux `
> to the test if that helps.
The problem here is with expensive checks, which makes it take too long I think. It works fine on Darwin bots without expensive checks. If it is not possible to reduce the size of the test, then we need to disable it at least for the configuration used for http://green.lab.llvm.org/green/job/clang-stage1-cmake-RA-expensive. I am not sure if there's another way to force a big gap in PPC assembly.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D86879/new/
https://reviews.llvm.org/D86879
More information about the llvm-commits
mailing list