This is an archive of the discontinued LLVM Phabricator instance.

dwp section overflow checks
AbandonedPublic

Authored by dblaikie on Mar 17 2023, 5:19 AM.

Details

Summary

This is a rebase to main branch of https://reviews.llvm.org/D144565. In addition to that patch, I added another test, checking debug_str section.

Diff Detail

Event Timeline

zhuna8616 created this revision.Mar 17 2023, 5:19 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 17 2023, 5:19 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
zhuna8616 requested review of this revision.Mar 17 2023, 5:19 AM
Herald added a project: Restricted Project. · View Herald Transcript
zhuna8616 edited the summary of this revision. (Show Details)Mar 17 2023, 5:28 AM
zhuna8616 added reviewers: ayermolo, steven.zhang.
This comment was removed by ayermolo.

I'm a bit confused - wasn't D144565 already committed? What's this then? Should this patch be abandoned now?

llvm/lib/DWP/DWP.cpp
722–724

(prefer = init over () init where both are valid, the former doesn't allow implicit conversions while the latter does - so seeing = can be easier to read because there's no explicit conversions (which can be more complicated/nuanced) to worry about) - and prefer declaring the variable in the smallest scope, including in the if condition where possible)

Yes. This one should be abandoned.

dblaikie commandeered this revision.Jun 6 2023, 1:50 PM
dblaikie abandoned this revision.
dblaikie added a reviewer: zhuna8616.