[llvm] [AArch64] Add support for SHF_AARCH64_PURECODE ELF section flag (1/3) (PR #125687)

[James Henderson via llvm-commits]

=?utf-8?q?Csanád_Hajdú?= <csanad.hajdu at arm.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/125687 at github.com>


================
@@ -1337,6 +1337,10 @@ const EnumEntry<unsigned> ElfXCoreSectionFlags[] = {
   ENUM_ENT(XCORE_SHF_DP_SECTION, "")
 };
 
+const EnumEntry<unsigned> ElfAArch64SectionFlags[] = {
+    ENUM_ENT(SHF_AARCH64_PURECODE, "y"),
----------------
jh7370 wrote:

That latter case doesn't look right to me at all and would suggest a clang-format bug, or at a minimum a missing feature, to me. Perhaps worth raising a clang-format issue to get the tool maintainers' opinion? I assume you're using a reasonably recent clang-format of course...

In the meantime, the general policy is "do what other code does in the area", so two spaces indentation and no comma, I presume.

https://github.com/llvm/llvm-project/pull/125687