This is an archive of the discontinued LLVM Phabricator instance.

[mlir] NFC - Move Vector structure from RunnerUtils.h to CRunnerUtils.h
ClosedPublic

Authored by nicolasvasilache on Feb 29 2020, 10:41 AM.

Details

Summary

The Vector struct does not require a C++ runtime.

Diff Detail

Event Timeline

ftynse accepted this revision.Mar 2 2020, 2:17 AM
ftynse added a subscriber: ftynse.

LGTM if you comply with clang-format. (Literally copy-pasting the code should be fine, don't know why you decided to reformat it).

This revision is now accepted and ready to land.Mar 2 2020, 2:17 AM

I did encounter a few cases where my clang-format does not agree with what phab says.
I rebuilt clang at HEAD today and I have:

.../llvm-project$ git-clang-format --binary=./build/bin/clang-format --style=LLVM HEAD^
clang-format did not modify any files

I can manually "fix" the format to make phab happy like I did in the past but I doubt phab is right.

ftynse added a comment.Mar 2 2020, 6:47 AM

I rebuilt clang at HEAD today and I have:

I think we are running some stable version of clang-format. @mehdi_amini for more context.

This revision was automatically updated to reflect the committed changes.