MaskRay added a subscriber: hans. MaskRay added a comment. In D72222#1836409 <https://reviews.llvm.org/D72222#1836409>, @peter.smith wrote:
> Although this particular commit will not be at fault, it is the option that > enables the feature which is the earliest I can bisect the fault to. There > are 3 files in linux that assert fail on the Implement the > 'patchable-function attribute'. The files in question are kasan/quarantine.c, > mm/slab_common.c and mm/slub.c . > > I reproduced with > > make CC=/path/to/clang/install/clang ARCH=arm64 > CROSS_COMPILE=aarch64-linux-gnu- HOSTGCC=gcc allmodconfig > > > You can get the log files for the build, which is from clang-10 > > https://git.linaro.org/toolchain/ci/base-artifacts.git/log/?h=linaro-local/ci/tcwg_kernel/llvm-release-aarch64-mainline-allmodconfig > (4: update: llvm-linux: 19676) > > If the patchable functions is intended for clang-10 we'll need to make sure > any fix is merged to clang-10. This commit was made before release/10.x branch. Maybe the easiest thing is to revert the driver change in release/10.x (CC @hans), before we had a better understanding of the problem. (Eventually I think the Linux kernel should have a better configure time test than a simple `whether the compiler accepts -fpatchable-function-entry=2,0?`) @peter.smith @nickdesaulniers What do you think? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72222/new/ https://reviews.llvm.org/D72222 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
