This is an archive of the discontinued LLVM Phabricator instance.

[nfc] [lldb] Fix harmless slicing of DWARFDIE
ClosedPublic

Authored by jankratochvil on Oct 21 2020, 5:54 AM.

Details

Summary

Is return {...}; implicit ctor OK for LLDB? I have found only one line (return {sdk};) in rG5935227e11f5 using that syntax and I would normally use it everywhere.

Diff Detail

Event Timeline

jankratochvil created this revision.Oct 21 2020, 5:54 AM
jankratochvil requested review of this revision.Oct 21 2020, 5:54 AM
labath accepted this revision.Oct 21 2020, 6:29 AM

Is return {...}; implicit ctor OK for LLDB? I have found only one line (return {sdk};) in rG5935227e11f5 using that syntax and I would normally use it everywhere.

We don't have a policy on that. I personally wouldn't recommend using it literally _everywhere_ (this can get messy in very long function and particularly if you start nesting the {}s), but here it's perfectly fine.

This revision is now accepted and ready to land.Oct 21 2020, 6:29 AM
This revision was landed with ongoing or failed builds.Oct 21 2020, 6:50 AM
This revision was automatically updated to reflect the committed changes.