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

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 5 18:40:33 PDT 2018


LGTM

On Wed, Sep 5, 2018, 3:41 PM Ivan Lozano via Phabricator <
reviews at reviews.llvm.org> wrote:

> ivanlozano updated this revision to Diff 164116.
> ivanlozano marked an inline comment as done.
> ivanlozano added a comment.
>
> Added additional comments to clarify to the source code additions.
>
>
> https://reviews.llvm.org/D48792
>
> Files:
>   include/llvm/MC/MCELFObjectWriter.h
>   include/llvm/MC/MCSection.h
>   lib/MC/ELFObjectWriter.cpp
>   lib/MC/MCELFObjectTargetWriter.cpp
>   lib/MC/MCObjectStreamer.cpp
>   lib/MC/MCSection.cpp
>   lib/Target/ARM/ARMTargetObjectFile.cpp
>   lib/Target/ARM/MCTargetDesc/ARMELFObjectWriter.cpp
>   test/CodeGen/ARM/execute-only.ll
>   test/MC/ARM/elf-execute-only-section.ll
>   test/MC/ELF/ARM/execute-only-populated-text-section.s
>   test/MC/ELF/ARM/execute-only-section.s
>   test/MC/ELF/ARM/execute-only-text-section-data.s
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180905/b1071fc2/attachment.html>


More information about the llvm-commits mailing list