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.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
This comment was removed by ayermolo.
Comment Actions
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) |
(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)