This is an archive of the discontinued LLVM Phabricator instance.

[clang][Diagnostic] Simplify emitDiagnosticLoc
ClosedPublic

Authored by tbaeder on May 22 2023, 6:32 AM.

Details

Summary
We don't use the offset returned from SourceManager::getDecomposedLoc
here, so we might as well just use getFileID().

Diff Detail

Event Timeline

tbaeder created this revision.May 22 2023, 6:32 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 22 2023, 6:32 AM
tbaeder requested review of this revision.May 22 2023, 6:32 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 22 2023, 6:32 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
aaron.ballman accepted this revision.May 22 2023, 9:10 AM

LGTM, though please add NFC to the patch summary when landing.

This revision is now accepted and ready to land.May 22 2023, 9:10 AM
This revision was landed with ongoing or failed builds.May 31 2023, 12:50 AM
This revision was automatically updated to reflect the committed changes.