This is an archive of the discontinued LLVM Phabricator instance.

[lldb] Don't use __FUNCTION__ as a file name
ClosedPublic

Authored by teemperor on Jul 10 2019, 10:41 PM.

Details

Summary

I saw while debugging that we call this file ParseInternal, which is not a very good name for our
fake expression file and also adds this unnecessary link between the way we name this function
and the other source location names we get from the expression parser. This patch is renaming
it to <lldb-expr> which is closer to the way Clang names its buffers, it doesn't depend on the
function name (which changes when I refactor this code) and it's easier to grep for.

Diff Detail

Repository
rL LLVM

Event Timeline

teemperor created this revision.Jul 10 2019, 10:41 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 10 2019, 10:41 PM
davide accepted this revision.Jul 11 2019, 7:40 AM

This seems fairly straighforward and I'm in favor of it. We maybe should do the same thing with swift, I haven't checked in a while how we call the file there.
I wouldn't bother adding a test for something so minor.

This revision is now accepted and ready to land.Jul 11 2019, 7:40 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptJul 11 2019, 12:27 PM