Details
Details
Diff Detail
Diff Detail
- Repository
 - rL LLVM
 
Event Timeline
Comment Actions
So much red, so pretty!
| lib/MC/MCParser/AsmParser.cpp | ||
|---|---|---|
| 700 ↗ | (On Diff #153401) | This one should be renamed, since there is now only one MCAsmParserExtension for wasm, and its implementation is actually kinda ELF-y, but it is wha we'll use for wasm mode going forward.  | 
| lib/Target/WebAssembly/WebAssemblyAsmPrinter.cpp | ||
| 188 ↗ | (On Diff #153401) | does it make sense to keep an empty function here for wasm functionality?  | 
| test/CodeGen/WebAssembly/negative-base-reg.ll | ||
| 13 ↗ | (On Diff #153401) | this I don't understand, how does your change affect it being written to a local or the stack?  | 
| lib/MC/MCParser/AsmParser.cpp | ||
|---|---|---|
| 700 ↗ | (On Diff #153401) | Maybe a separate change?  | 
| lib/MC/MCParser/AsmParser.cpp | ||
|---|---|---|
| 700 ↗ | (On Diff #153401) | that's fine with me.  |