This is an archive of the discontinued LLVM Phabricator instance.

[mlir] tosa: build error when building with clang
ClosedPublic

Authored by lipracer on Mar 28 2022, 10:35 AM.

Details

Summary

The change of https://reviews.llvm.org/D121513#3411651
has caused a build error when building with clang:

/mnt/vss/_work/1/llvm-project/mlir/lib/Dialect/Tosa/IR/TosaOps.cpp:599:26: error: extra ';' outside of a function is incompatible with C++98 [-Werror,-Wc++98-compat-extra-semi]
ReduceFolder(ReduceAllOp);

Diff Detail

Event Timeline

lipracer created this revision.Mar 28 2022, 10:35 AM
Herald added a project: Restricted Project. · View Herald Transcript
lipracer requested review of this revision.Mar 28 2022, 10:35 AM
hpmorgan accepted this revision.Mar 28 2022, 10:39 AM
hpmorgan added a subscriber: hpmorgan.

LGTM. Thanks!

This revision is now accepted and ready to land.Mar 28 2022, 10:39 AM

@hpmorgan Could you please review it, It's very strange why we get normal clang-format results when we change the macro definition to uppercase.

Mogball accepted this revision.Mar 28 2022, 12:59 PM

I'll land this one

This revision was landed with ongoing or failed builds.Mar 28 2022, 1:02 PM
This revision was automatically updated to reflect the committed changes.