MaskRay added a comment.
I have a question why the option is needed.
if (IBTSeal) {
return F.hasAddressTaken();
}
// if !IBTSeal, fall into default case.
LLVM_FALLTHROUGH;
// Address taken or externally linked functions may be reachable.
default:
return (F.hasAddressTaken() || !F.hasLocalLinkage());
`F.hasAddressTaken() || !F.hasLocalLinkage()` can be changed to
`F.hasAddressTaken() || (F.hasHiddenVisibility() && !F.hasLocalLinkage())`
Why can't the kernel use `-fvisibility=hidden`?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116070/new/
https://reviews.llvm.org/D116070
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
