[PATCH] D88815: Added .tabletype to asm and multiple table support in obj files

Tue Oct 13 02:31:35 PDT 2020

pmatos added a comment.

In D88815#2326785 <https://reviews.llvm.org/D88815#2326785>, @pmatos wrote:

> In D88815#2326737 <https://reviews.llvm.org/D88815#2326737>, @sbc100 wrote:
>
>> I seeing:
>>
>>   llvm-project/llvm/lib/Target/WebAssembly/WebAssemblyAsmPrinter.cpp:111:11: warning: enumeration value 'FUNCREF' not handled in switch [-Wswitch]
>>     switch (VT) {
>
> Oh, I was not getting that. It seems to come from https://github.com/llvm/llvm-project/commit/3bba91f64eef15956f589fa446c265a714cc7893 and I hadn't pulled that in yet. If I rebase my patch on this then yes, I can see the warning. i will fix it in this patch. No worries.

All looks good now.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88815