atrosinenko added a comment.
@rjmccall Uploaded, thank you!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85113/new/
https://reviews.llvm.org/D85113
___
cfe-commits mailing list
cfe-commits@lists.llvm.or
This revision was automatically updated to reflect the committed changes.
Closed by commit rG5a07490d7611: [ABI][NFC] Fix the confusion of ByVal and
ByRef argument names (authored by atrosinenko).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85113/n
rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85113/new/
https://reviews.llvm.org/D85113
_
atrosinenko created this revision.
atrosinenko added reviewers: majnemer, rjmccall, jpienaar.
Herald added a project: clang.
atrosinenko requested review of this revision.
The second argument of getNaturalAlignIndirect() was `bool ByRef`, but
the implementation was just delegating to getIndirect()