This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Comment out a switch block in ISelDAGToDAG
ClosedPublic

Authored by aheejin on Jun 28 2018, 1:05 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

aheejin created this revision.Jun 28 2018, 1:05 PM
aheejin edited the summary of this revision. (Show Details)Jun 28 2018, 1:08 PM
RKSimon accepted this revision.Jun 29 2018, 6:07 AM

LGTM with one minor

lib/Target/WebAssembly/WebAssemblyISelDAGToDAG.cpp
81 ↗(On Diff #153377)

Probably not worth keeping the VT here? Its easy enough to add back if you need it - I think you're mainly interested in showing how to use the switch statement.

This revision is now accepted and ready to land.Jun 29 2018, 6:07 AM
aheejin updated this revision to Diff 153570.Jun 29 2018, 2:18 PM
aheejin marked an inline comment as done.
  • Removed VT
This revision was automatically updated to reflect the committed changes.