This is an archive of the discontinued LLVM Phabricator instance.

[VectorOps] Fixed typo in lowering code
AbandonedPublic

Authored by aartbik on Jan 13 2020, 3:39 PM.

Details

Reviewers
nicolasvasilache
Summary

(also first test CL using phabricator)

Diff Detail

Event Timeline

aartbik created this revision.Jan 13 2020, 3:39 PM
nicolasvasilache requested changes to this revision.Jan 13 2020, 3:47 PM
nicolasvasilache added inline comments.
mlir/lib/Conversion/VectorToLLVM/ConvertVectorToLLVM.cpp
27

Note: we use llvm::errs/outs/dbgs consistently across the codebase, not iostream

This revision now requires changes to proceed.Jan 13 2020, 3:47 PM
aartbik abandoned this revision.Jan 13 2020, 3:48 PM

This uploaded a default CL not the required CL. Abandoning.....

Unit tests: pass. 61744 tests passed, 0 failed and 780 were skipped.

clang-tidy: unknown.

clang-format: pass.

Build artifacts: diff.json, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml