This is an archive of the discontinued LLVM Phabricator instance.

[clang][NFC] Fix needless double-parenthisation
ClosedPublic

Authored by wingo on Aug 19 2021, 2:57 AM.

Details

Summary

Strip a layer of parentheses in TreeTransform::RebuildQualifiedType.

Diff Detail

Event Timeline

wingo requested review of this revision.Aug 19 2021, 2:57 AM
wingo created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptAug 19 2021, 2:57 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript

Hi Richard, I am new to clang, having mostly worked on the WebAssembly target in llvm proper -- I have a stack of patches, of which a couple later ones touch Sema/. See https://lists.llvm.org/pipermail/cfe-dev/2021-July/068559.html for broader context. However I had a couple small NFC cleanups I found when reading code, like this one; would it be OK if I sent them your way? Cheers, Andy.

Just a little thing I saw while reading code -- if no response in a couple weeks I will just abandon the rev

Anastasia accepted this revision.Sep 21 2021, 3:44 AM

LGTM! Thanks

This revision is now accepted and ready to land.Sep 21 2021, 3:44 AM
This revision was automatically updated to reflect the committed changes.