This is an archive of the discontinued LLVM Phabricator instance.

Fixed a typo in the comment for allocateBuffer()
ClosedPublic

Authored by georgemitenkov on May 17 2020, 9:36 AM.

Details

Summary

This commit fixes a typo in the comment and is intended to make sure development flow works

Diff Detail

Event Timeline

georgemitenkov created this revision.May 17 2020, 9:36 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 17 2020, 9:36 AM
mehdi_amini accepted this revision.May 17 2020, 10:15 AM
mehdi_amini added inline comments.
mlir/lib/Conversion/StandardToLLVM/StandardToLLVM.cpp
1

Might be useful to install clang-format on your development machine for future patches.

This revision is now accepted and ready to land.May 17 2020, 10:15 AM
antiagainst accepted this revision.May 18 2020, 11:37 AM

Awesome! It's great to see your first path, George! :) I'll land this.

mlir/lib/Conversion/StandardToLLVM/StandardToLLVM.cpp
1

+1. Also you can run git clang-format to format the code.

This revision was automatically updated to reflect the committed changes.