Please use GitHub pull requests for new patches. Phabricator shutdown timeline

[mlir][sparse] Fixing -Wignored-reference-qualifiers in MergerTest.cpp
ClosedPublic
Actions

Authored by wrengr on Mar 28 2023, 2:48 PM.

Details

Reviewers

aartbik
bixia
Peiming
nicolasvasilache

Commits

rG0ceec3ebfcf3: [mlir][sparse] Fixing -Wignored-reference-qualifiers in MergerTest.cpp

Summary

These warnings were introduced by D146561.

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

wrengr created this revision.Mar 28 2023, 2:48 PM

Herald added a project: Restricted Project. · View Herald TranscriptMar 28 2023, 2:48 PM

Herald added subscribers: hanchung, jsetoain, Moerafaat and 20 others. · View Herald Transcript

wrengr requested review of this revision.Mar 28 2023, 2:48 PM

Herald added a reviewer: nicolasvasilache. · View Herald TranscriptMar 28 2023, 2:48 PM

Herald added a project: Restricted Project. · View Herald Transcript

Herald added subscribers: stephenneuendorffer, nicolasvasilache. · View Herald Transcript

Peiming accepted this revision.Mar 28 2023, 3:54 PM

This revision is now accepted and ready to land.Mar 28 2023, 3:54 PM

aartbik accepted this revision.Mar 28 2023, 3:58 PM

rebase

This revision was landed with ongoing or failed builds.Mar 28 2023, 4:33 PM

Closed by commit rG0ceec3ebfcf3: [mlir][sparse] Fixing -Wignored-reference-qualifiers in MergerTest.cpp (authored by wrengr). · Explain Why

This revision was automatically updated to reflect the committed changes.

wrengr added a commit: rG0ceec3ebfcf3: [mlir][sparse] Fixing -Wignored-reference-qualifiers in MergerTest.cpp.

Harbormaster completed remote builds in B222373: Diff 509169.Mar 29 2023, 1:16 AM

Revision Contents
Changeset List

Path

Size

mlir/

unittests/

Dialect/

SparseTensor/

MergerTest.cpp

46 lines

Diff 509169

View Options

mlir/unittests/Dialect/SparseTensor/MergerTest.cpp