[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>


================
@@ -1375,6 +1379,10 @@ getSectionFlagsForTarget(unsigned EOSAbi, unsigned EMachine) {
     break;
   }
   switch (EMachine) {
+  case EM_AARCH64:
----------------
jh7370 wrote:

The llvm-readobj test case should use yaml2obj to generate the input from a YAML file, so that way the test works without any MC or CodeGen changes. Indeed, it's quite normal for dumper tool support to be added independently and before the assembler/code generation level support (I'm not advocating splitting it at this point, but that might help clarify the testing responsibility).

Also, tests for llvm-readobj behaviour belong in the test/tools/llvm-readobj directory, to help identify coverage. I expect there are equivalent tests for other platforms doing the same thing there, which might help clarify what is expected.

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