This is an archive of the discontinued LLVM Phabricator instance.

Update Kaleidoscope tutorial and remove useless header
ClosedPublic

Authored by achieveartificialintelligence on Sep 23 2020, 3:05 AM.

Details

Summary

[Kaleidoscope]
Update Kaleidoscope tutorial and remove useless header

Some header files are no longer used, remove them can speed up compilation.

Patch by: Mii Muu <miimuu@miimuu.com>

Diff Detail

Event Timeline

achieveartificialintelligence requested review of this revision.Sep 23 2020, 3:05 AM

Looks OK for me, but utility should be also included because of std::move.

In future I could recommend to use Include What You Use. Will be good idea to do similar clean-ups for other examples.

thanks.

added <utility>

JDevlieghere accepted this revision.Sep 23 2020, 3:41 PM
This revision is now accepted and ready to land.Sep 23 2020, 3:41 PM
This revision was automatically updated to reflect the committed changes.