[PATCH] D48792: [ARM] Set execute-only flags in .text.

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 5 14:45:44 PDT 2018


echristo added inline comments.


================
Comment at: lib/MC/MCObjectStreamer.cpp:500
+  MCSection *Sec = getCurrentSectionOnly();
+  Sec->setHasData(true);
 }
----------------
Given that the only target that cares about hasData is the ARM backend you could probably override this there and only worry about setting it within that backend?


https://reviews.llvm.org/D48792





More information about the llvm-commits mailing list