This is an archive of the discontinued LLVM Phabricator instance.

[WebAssembly] Use Function::hasOptSize() (NFC)
ClosedPublic

Authored by uenoku on Apr 13 2019, 7:14 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

uenoku created this revision.Apr 13 2019, 7:14 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 13 2019, 7:14 AM
aheejin accepted this revision.Apr 13 2019, 9:17 AM

Thanks! Could you append (NFC) to the CL title?

llvm/lib/Target/WebAssembly/WebAssemblyISelDAGToDAG.cpp
55 ↗(On Diff #195014)

I think we can delete this newly added newline?

This revision is now accepted and ready to land.Apr 13 2019, 9:17 AM
uenoku updated this revision to Diff 195017.Apr 13 2019, 9:21 AM

Remove newline

uenoku marked an inline comment as done.Apr 13 2019, 9:21 AM
uenoku retitled this revision from [WebAssembly] Use Function::hasOptSize() to [WebAssembly] Use Function::hasOptSize() (NFC).

I don't have commit access. Would you commit?

This revision was automatically updated to reflect the committed changes.