[PATCH] D48792: [ARM] Set execute-only flags in .text.
Ivan Lozano via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 3 08:55:56 PDT 2018
ivanlozano added a comment.
Yea I wasn't sure exactly which approach to take here. Initially I thought that setting the TextSection to always include SHF_ARM_PURECODE was the way to go since I hadn't seen that section being used when generating object files from source. However I discovered from running the test suite that this section is used when assembling machine code and outputting an object-file with llvm-mc. This erroneously set the SHF_ARM_PURECODE flag when no execute-only flag was provided. This leads me to believe there may be other scenarios where this section is used as well.
With regard to the LTO question, I'm unfamiliar with how it works so that's not something I've considered. I am definitely open to suggestions if we need to avoid the Subtarget lookup.
Repository:
rL LLVM
https://reviews.llvm.org/D48792
More information about the llvm-commits
mailing list