[PATCH] D96473: [WebAssembly] Fix assert in lookup of section symbols

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 11 16:40:55 PST 2021


sbc100 accepted this revision.
sbc100 added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/test/MC/WebAssembly/section-symbol.s:9
+    .section    .debug_info,"",@
+    .int32      .debug_abbrev0
+
----------------
This does seem rather odd.   It seems like this should be a error... if we are going to add a test for this seemly broken behaviour, perhaps we should add a TODO with a link to a bug to address it?


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

https://reviews.llvm.org/D96473



More information about the llvm-commits mailing list