joaomoreira added a comment.

Regarding https://github.com/ClangBuiltLinux/linux/issues/1737:

Weirdly enough, I double-tested the behavior for -flto=thin + -mibt-seal; the 
kernel did boot fine on my setup, but when dumped/grep'ed for ENDBRs, it had 
~500 less ENDBRs throughout the binary. Considering that LTO-based 
optimizations applied with -flto=thin should have also been applied with -flto 
(which means, things like dead-code elimination should be symmetrical in both 
generated binaries), and also given the confirmation that -flto=thin won't 
properly keep address-taken details across translation units, my understanding 
is that -mibt-seal must be disabled when -flto=thin. Thus, submitted the patch 
even though I was not able to reproduce the bad behavior.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140035

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to