[PATCH] D128121: [NFC][Alignment] Remove max functions between Align and MaybeAlign

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 18 07:37:33 PDT 2022


gchatelet created this revision.
gchatelet added a reviewer: courbet.
Herald added subscribers: kosarev, jsilvanus, mattd, gchakrabarti, hsmhsm, asavonic, ormris, foad, okura, jdoerfert, kuter, kerbowa, steven_wu, hiraditya, nhaehnle, jvesely, arsenm.
Herald added a project: All.
gchatelet requested review of this revision.
Herald added subscribers: llvm-commits, jholewinski.
Herald added a reviewer: jdoerfert.
Herald added a reviewer: sstefan1.
Herald added a project: LLVM.

`llvm::max(Align, MaybeAlign)` and `llvm::max(MaybeAlign, Align)` are
not used often enough to be required. They also make the code more opaque.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D128121

Files:
  llvm/include/llvm/Support/Alignment.h
  llvm/lib/LTO/LTO.cpp
  llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp
  llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp
  llvm/lib/Target/Hexagon/HexagonVExtract.cpp
  llvm/lib/Target/NVPTX/NVPTXAsmPrinter.cpp
  llvm/lib/Transforms/IPO/AttributorAttributes.cpp
  llvm/lib/Transforms/Utils/InlineFunction.cpp
  llvm/lib/Transforms/Utils/MemoryTaggingSupport.cpp
  llvm/tools/llvm-lto2/llvm-lto2.cpp
  llvm/unittests/Support/AlignmentTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D128121.438126.patch
Type: text/x-patch
Size: 8615 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220618/016e55e0/attachment.bin>


More information about the llvm-commits mailing list