[PATCH] D120305: [Driver] Default CLANG_DEFAULT_PIE_ON_LINUX to ON

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 25 16:43:36 PST 2022


MaskRay added a comment.

In D120305#3346787 <https://reviews.llvm.org/D120305#3346787>, @nemanjai wrote:

> In D120305#3346144 <https://reviews.llvm.org/D120305#3346144>, @MaskRay wrote:
>
>> In D120305#3345978 <https://reviews.llvm.org/D120305#3345978>, @MaskRay wrote:
>>
>>> In D120305#3345810 <https://reviews.llvm.org/D120305#3345810>, @RKSimon wrote:
>>>
>>>> @MaskRay The ppc buildbots have been red since these patches - please can you take a look? https://lab.llvm.org/buildbot/#/builders/57/builds/15454
>>>
>>> Seems that ppc64 doesn't support test/sanitizer_common test/crt -fpie. I'll just disable them: https://github.com/llvm/llvm-project/issues/54084
>>
>> Actually I don't know how to disable the tests: https://lab.llvm.org/buildbot/#/builders/57/builds/15497 still failed.
>> Hope someone from #powerpc <https://reviews.llvm.org/tag/powerpc/> can disable them.
>
> This does not appear to be a matter of simply marking some tests as UNSUPPORTED. Since this landed, there have been many builds with different sanitizer failures and different numbers of sanitizer failures. Please pull this patch to bring the bots back to green and we can work with you next week on fixing what needs to be fixed.

I enabled -DCLANG_DEFAULT_PIE_ON_LINUX=OFF for clang-ppc64le-rhel: https://github.com/llvm/llvm-zorg/commit/b6ddf02ce3a54da2df29e7e599b1838167e0e3ad
which should fix the issues.

While I feel sorry for leaving clang-ppc64le-rhel red for some time and am willing to fix issues if I have access to a ppc64 machine (especially compiler-rt ones that I care about),
I feel uncomfortably if a group just bluntly request "please pull this patch" when apparently (a) there is a better approach (explicitly setting CLANG_DEFAULT_PIE_ON_LINUX=OFF) and (b) there is something a bot maintainer can do
and (c) there is just some inherent stability problem (in this case, consider not enabling the testing when the target is still unstable).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120305



More information about the cfe-commits mailing list