diff options
| author | Guillaume Chatelet <gchatelet@google.com> | 2020-01-21 15:00:04 +0100 |
|---|---|---|
| committer | Guillaume Chatelet <gchatelet@google.com> | 2020-01-24 12:53:58 +0100 |
| commit | 805c157e8aeee24adec897bed42662733eb0d710 (patch) | |
| tree | 23f9e5adbcc7b67a96d72e13d2e2714ebdb00f2e /llvm/lib/CodeGen/MachineOperand.cpp | |
| parent | 2ee218f3653fcfd135fc5087a698424f90a9a499 (diff) | |
| download | llvm-805c157e8aeee24adec897bed42662733eb0d710.zip llvm-805c157e8aeee24adec897bed42662733eb0d710.tar.gz llvm-805c157e8aeee24adec897bed42662733eb0d710.tar.bz2 | |
[Alignment][NFC] Deprecate Align::None()
Summary:
This is a follow up on https://reviews.llvm.org/D71473#inline-647262.
There's a caveat here that `Align(1)` relies on the compiler understanding of `Log2_64` implementation to produce good code. One could use `Align()` as a replacement but I believe it is less clear that the alignment is one in that case.
Reviewers: xbolva00, courbet, bollu
Subscribers: arsenm, dylanmckay, sdardis, nemanjai, jvesely, nhaehnle, hiraditya, kbarton, jrtc27, atanasyan, jsji, Jim, kerbowa, cfe-commits, llvm-commits
Tags: #clang, #llvm
Differential Revision: https://reviews.llvm.org/D73099
Diffstat (limited to 'llvm/lib/CodeGen/MachineOperand.cpp')
| -rw-r--r-- | llvm/lib/CodeGen/MachineOperand.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/llvm/lib/CodeGen/MachineOperand.cpp b/llvm/lib/CodeGen/MachineOperand.cpp index 0614ab5..c7cbd9e 100644 --- a/llvm/lib/CodeGen/MachineOperand.cpp +++ b/llvm/lib/CodeGen/MachineOperand.cpp @@ -970,8 +970,7 @@ bool MachinePointerInfo::isDereferenceable(unsigned Size, LLVMContext &C, return false; return isDereferenceableAndAlignedPointer( - BasePtr, Align::None(), APInt(DL.getPointerSizeInBits(), Offset + Size), - DL); + BasePtr, Align(1), APInt(DL.getPointerSizeInBits(), Offset + Size), DL); } /// getConstantPool - Return a MachinePointerInfo record that refers to the |