[PATCH] D59780: Support Intel Control-flow Enforcement Technology

Xiang Zhang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 28 20:09:59 PDT 2019


xiangzhangllvm added a comment.

In D59780#1520411 <https://reviews.llvm.org/D59780#1520411>, @MaskRay wrote:

> In D59780#1520390 <https://reviews.llvm.org/D59780#1520390>, @xiangzhangllvm wrote:
>
> > I'm sorry, I reflect the problem to some leaders and experts, but the PLT ABI is still undecided. The most concern come from the unknow risk of difference with GNU.
>
>
> @xiangzhangllvm If it is still undecided, then there is still opportunity to fix :) IMHO tooling is still immature and currently very few tools understand `.plt.sec`.
>
> I'd really like to see more discussions about this, e.g. https://groups.google.com/forum/#!topic/x86-64-abi/Z6GqQDy_NaI
>  If PLT entries should be aligned by 16 to maximize performance, shall we switch to 32-byte PLT entry? Xiang, can you raise a topic on https://sourceware.org/ml/gnu-gabi/2019-q2 or https://sourceware.org/ml/binutils/2019-05/


hi, fangrui, 32-byte PLT entry is also aligned by 16, I have test the performance before, they are same.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D59780





More information about the llvm-commits mailing list