[PATCH] D55401: [WebAssembly] Fix assembler parsing of br_table.
Derek Schuff via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 7 13:45:26 PST 2018
dschuff added inline comments.
================
Comment at: lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCCodeEmitter.cpp:97
if (i < Desc.getNumOperands()) {
- assert(Desc.TSFlags == 0 &&
+ assert((Desc.TSFlags == 0 || IsBrTable) &&
"WebAssembly non-variable_ops don't use TSFlags");
----------------
aardappel wrote:
> This is pretty yucky. I must I don't quite follow what these flags are used for, so not sure if we should remove these flags from br_table in the tablegen defs instead?
I think we can remove the flags from br_table in tablegen because it doesn't use variable_ops anymore. I think they are only used for determining how the variable_ops should be printed.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55401/new/
https://reviews.llvm.org/D55401
More information about the llvm-commits
mailing list