[PATCH] D85774: [XCOFF][AIX] Enable tooling support for 64 bit symbol table parsing

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 2 12:05:18 PDT 2020


MaskRay added inline comments.


================
Comment at: llvm/include/llvm/Object/XCOFFObjectFile.h:145
+  uint32_t getSectionOrLength32() const {
+    assert(Entry32 && "32-bit interface called on 64-bit object file.");
+    return Entry32->SectionOrLength;
----------------
You probably don't need these assert. The dereference will crash anyway


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

https://reviews.llvm.org/D85774



More information about the llvm-commits mailing list