This is an archive of the discontinued LLVM Phabricator instance.

[mlir] Update Location to use new casting infra
ClosedPublic

Authored by nkreeger on Oct 21 2022, 8:58 PM.

Details

Summary

This allows for using the llvm namespace cast methods instead of the ones on the Location class. The Location class method are kept for now, but we'll want to remove these eventually (with a really long lead time).

Related change: https://reviews.llvm.org/D135870

Diff Detail

Event Timeline

nkreeger created this revision.Oct 21 2022, 8:58 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 21 2022, 8:58 PM
nkreeger requested review of this revision.Oct 21 2022, 8:58 PM
nkreeger updated this revision to Diff 469861.Oct 21 2022, 9:01 PM
rriddle accepted this revision.Oct 21 2022, 9:03 PM
rriddle added inline comments.
mlir/include/mlir/IR/Location.h
187

Do we need the base_of? I don't think anything inherits from Location.

This revision is now accepted and ready to land.Oct 21 2022, 9:03 PM
nkreeger updated this revision to Diff 469886.Oct 22 2022, 5:48 AM
nkreeger marked an inline comment as done.
This revision was automatically updated to reflect the committed changes.