[PATCH] D72222: [Driver][CodeGen] Add -fpatchable-function-entry=N[,0]
Peter Smith via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 23 10:19:00 PST 2020
peter.smith added a comment.
>> 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?
Revert on the 10.0 release sounds reasonable to me. That would prevent the kernel from enabling the option and would prevent the build failure.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72222/new/
https://reviews.llvm.org/D72222
More information about the cfe-commits
mailing list