[PATCH] D144083: [JITLink] Initial AArch32 backend

[Stefan Gränitz via Phabricator via llvm-commits]

sgraenitz marked an inline comment as not done.
sgraenitz added inline comments.


================
Comment at: llvm/lib/ExecutionEngine/JITLink/ELFLinkGraphBuilder.h:113
+  /// graph.
+  virtual bool excludeSectionByName(StringRef Name) const { return false; }
+
----------------
lhames wrote:
> Is there an ELF attribute or type that we could be looking at, rather than the name?
> 
> (I'm very guilty of using section names rather than attributes in checks, but as JITLink evolves I do want us to use attributes and types more where possible)
Yes this was a quick one. Let me check if I find something better. This is still open.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144083/new/

https://reviews.llvm.org/D144083