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

Wouter van Oortmerssen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 18 11:45:37 PST 2021


aardappel marked an inline comment as done.
aardappel added inline comments.


================
Comment at: llvm/test/MC/WebAssembly/section-symbol.s:9
+    .section    .debug_info,"",@
+    .int32      .debug_abbrev0
+
----------------
sbc100 wrote:
> 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?
created bug and TODO: https://bugs.llvm.org/show_bug.cgi?id=49252


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

https://reviews.llvm.org/D96473



More information about the llvm-commits mailing list