[PATCH] D33046: [ELF] - Set DF_TEXTREL flag properly.

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 10 09:06:53 PDT 2017


davide added a comment.

Side note: if everything works just fine as is (TM), why do we need to make this change?
It buys us a new shiny flag that is pretty much equivalent to the old one for systems supporting both the `DT_` and the `DF_` variant, but it potentially breaks backcompatibility with older loaders not supporting the new form.
I'm not entirely sure this risk is worth it.


https://reviews.llvm.org/D33046





More information about the llvm-commits mailing list