SuccessChanges

Changes from Git (git http://labmaster3.local/git/llvm-project.git)

Summary

  1. Remove strcmp fuzz from CMakeList since it breaks build. (details)
  2. [NewPM] Attempt to run opt passes specified via -foo-pass under NPM (details)
  3. [X86][AVX] Add v16f32 variant of PR46249 test case (details)
  4. Revert "[mlir] make the bitwidth of device side index computations configurable" (details)
  5. [X86][AVX] Attempt to lower v16i32/v16f32 shuffles with lowerShuffleAsRepeatedMaskAndLanePermute (details)
  6. [debugserver] Initial support for Apple Silicon. (details)
  7. [Coroutines] Fix a few failing tests (details)
  8. [mlir] Added the dialect inliner to the SCF dialect (details)
  9. [lldb][NFC] Add some more tests for edge cases LLDB's builtin formatters (details)
  10. [SVE] Remove calls to VectorType::getNumElements from FuzzMutate (details)
  11. Fix incorrect "REQUIRE" (default_target->default_triple) introduced in 59f45a1cdb3 (details)
  12. [MLIR][LLVMDialect] Added bitreverse and ctpop intrinsics (details)
  13. [vscode] set default values for terminateDebuggee for the disconnect request (details)
  14. [ELF] Resolve relocations in .debug_* referencing (discarded symbols or ICF folded section symbols) to tombstone values (details)
  15. Change CMake so that we only look for Z3 when LLVM_ENABLE_Z3_SOLVER is enabled (details)
  16. [AMDGPU/MemOpsCluster] Implement new heuristic for computing max mem ops cluster size (details)
  17. [mlir] Avoid pontentially ambiguous class name (details)
  18. [Host] Check for TARGET_OS_EMBEDDED instead of listing architectures. (details)
  19. [ObjectFileMachO] Check for TARGET_EMBEDDED instead of listing architectures. (details)
  20. Generalize TestFormattersBoolRefPtr to work on Apple Silicon. (details)
  21. Redo of Add terminateCommands to lldb-vscode protocol (details)
  22. [LLD][PowerPC] Add support for R_PPC64_PCREL34 (details)
  23. [DSE,MSSA] Treat `store 0` after calloc as noop stores. (details)
  24. [libc][Obvious] Fix few typos in tests. (details)
  25. [IR] Remove unnecessary uint64_t casts (NFC) (details)
Commit 8bec88e638481f9f9a1f2d1d3c2349d7bc366e9f by gyurgyikcp
Remove strcmp fuzz from CMakeList since it breaks build.
The file was modifiedlibc/fuzzing/string/CMakeLists.txt
Commit 23f5fd85bf9f30ea289ba8328ba6dc65bd1dc3d1 by aeubanks
[NewPM] Attempt to run opt passes specified via -foo-pass under NPM

Summary:
In order to enable mass testing of opt under NPM, specifically passes
specified via -foo-pass.

This is gated under a new opt flag -enable-new-pm.  Currently
the pass flag parser looks for legacy PM passes with the name "foo" (for
opt arg "-foo") and creates a PassInfo for each one. Here we take the
(legacy PM) pass name and try to match it with one defined in (NPM)
PassRegistry.def.  Ultimately if we want all tests to pass like this,
we'll need to port all passes to NPM and register them in
PassRegistry.def under the same name as they were reigstered in the
legacy PM.

Maybe at some point we'll migrate all -foo to --passes=foo, but that
would be after the NPM switch.

Flipping on the flag causes 2XXX failures under check-llvm. By far most
of them are passes either not ported to NPM or don't have the same name
in PassRegistry.def as their old name.

Reviewers: hans, echristo, asbirlea, leonardchan

Subscribers: llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D82320
The file was addedllvm/test/Other/opt-npm.ll
The file was modifiedllvm/tools/opt/NewPMDriver.h
The file was modifiedllvm/tools/opt/NewPMDriver.cpp
The file was modifiedllvm/tools/opt/opt.cpp
Commit f9b4b8e3b3d6e082b87a556805f3345f7f80965e by llvm-dev
[X86][AVX] Add v16f32 variant of PR46249 test case
The file was modifiedllvm/test/CodeGen/X86/vector-shuffle-512-v16.ll
Commit 2ff6fad70049b340eeb7cb281c1466fc0169fd17 by tobias.gysi
Revert "[mlir] make the bitwidth of device side index computations configurable"

This reverts commit d10b1a38a7dfb994623f27f263b67f5fc76e08cc.
The file was modifiedmlir/include/mlir/Conversion/StandardToLLVM/ConvertStandardToLLVM.h
The file was modifiedmlir/include/mlir/Conversion/StandardToLLVM/ConvertStandardToLLVMPass.h
The file was modifiedmlir/test/Conversion/GPUToNVVM/gpu-to-nvvm.mlir
The file was modifiedmlir/include/mlir/Conversion/GPUToNVVM/GPUToNVVMPass.h
The file was modifiedmlir/include/mlir/Conversion/GPUToROCDL/GPUToROCDLPass.h
The file was modifiedmlir/lib/Conversion/GPUToROCDL/LowerGpuOpsToROCDLOps.cpp
The file was modifiedmlir/lib/Conversion/StandardToLLVM/StandardToLLVM.cpp
The file was modifiedmlir/include/mlir/Conversion/Passes.td
The file was modifiedmlir/lib/Conversion/GPUToNVVM/LowerGpuOpsToNVVMOps.cpp
The file was modifiedmlir/test/Conversion/GPUToROCDL/gpu-to-rocdl.mlir
Commit e7e204a373683a09978be2b1df8783ba26fe43a4 by llvm-dev
[X86][AVX] Attempt to lower v16i32/v16f32 shuffles with lowerShuffleAsRepeatedMaskAndLanePermute

Avoids prematurely creating permps/permd variable shuffles.

Fixes PR46249
The file was modifiedllvm/lib/Target/X86/X86ISelLowering.cpp
The file was modifiedllvm/test/CodeGen/X86/vector-shuffle-512-v16.ll
Commit 2276bb48be0175f96b1494ca67b7921f0d4e87d8 by ditaliano
[debugserver] Initial support for Apple Silicon.

Set the correct os type in the arch triple when running macOS.
Debugserver currently always assumes macOS == x86_64. This patch
generalizes the support to make sure it works on a different
architecture.

Differential Revision:  https://reviews.llvm.org/D82394
The file was modifiedlldb/tools/debugserver/source/RNBRemote.cpp
Commit f2dd08844c1aefc167328704b69e187de6409632 by xun
[Coroutines] Fix a few failing tests

Summary:
https://reviews.llvm.org/D82029 introduced the non-throw check for final_suspend(). There are a few tests I missed in that patch.
Fixing them here.

Reviewers: #libc, lewissbaker, modocache, ldionne

Reviewed By: #libc, ldionne

Subscribers: dexonsmith, modocache, libcxx-commits

Tags: #libc

Differential Revision: https://reviews.llvm.org/D82338
The file was modifiedlibcxx/test/std/experimental/language.support/support.coroutines/end.to.end/go.pass.cpp
The file was modifiedlibcxx/test/std/experimental/language.support/support.coroutines/end.to.end/fullexpr-dtor.pass.cpp
The file was modifiedlibcxx/test/std/experimental/language.support/support.coroutines/end.to.end/multishot_func.pass.cpp
The file was modifiedlibcxx/test/std/experimental/language.support/support.coroutines/end.to.end/oneshot_func.pass.cpp
Commit 5048933c47157a55349e623c289c9ed77d58d628 by liufeng.ee
[mlir] Added the dialect inliner to the SCF dialect

Currently no restrictions are added to the destination regions.

Subscribers: mehdi_amini, rriddle, jpienaar, shauheen, antiagainst, nicolasvasilache, arpith-jacob, mgester, lucyrfox, aartbik, stephenneuendorffer, Joonsoo, grosul1, Kayjukh, jurahul, msifontes

Tags: #mlir

Differential Revision: https://reviews.llvm.org/D82336
The file was modifiedmlir/lib/Dialect/SCF/SCF.cpp
Commit d13c3e2f88c621d43b583e3040b127924bcebb3e by Raphael Isemann
[lldb][NFC] Add some more tests for edge cases LLDB's builtin formatters

OSType with less than 8 bytes has special code that isn't tested yet.
The same for C-strings that don't have `const char *` type. Also we're now testing
escaping the ASCII escape sequence (\033).
The file was modifiedlldb/test/API/functionalities/data-formatter/builtin-formats/TestBuiltinFormats.py
Commit 4d1fd33561cf758be00bdbffab1b6a1a0e428fc0 by ctetreau
[SVE] Remove calls to VectorType::getNumElements from FuzzMutate

Reviewers: efriedma, bkramer, kmclaughlin, sdesmalen

Reviewed By: sdesmalen

Subscribers: tschuett, hiraditya, rkruppe, psnobl, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D82212
The file was modifiedllvm/lib/FuzzMutate/Operations.cpp
Commit 8b64adb0a2853f42c09bd98759eec86f51b546ba by joker.eph
Fix incorrect "REQUIRE" (default_target->default_triple) introduced in 59f45a1cdb3

Adding `default_target` fixed the build by excluding these tests... but
this excluded these tests from ever running!
The correct feature check is `default_triple`
The file was modifiedllvm/test/Examples/Kaleidoscope/Chapter7.test
The file was modifiedllvm/test/Examples/Kaleidoscope/Chapter4.test
The file was modifiedllvm/test/Examples/Kaleidoscope/Chapter5.test
The file was modifiedllvm/test/Examples/Kaleidoscope/Chapter6.test
Commit a2edbd8170bd55ea64466a0719e4adda4b08a195 by antiagainst
[MLIR][LLVMDialect] Added bitreverse and ctpop intrinsics

Introduced `llvm.intr.bitreverse` and `llvm.intr.ctpop` LLVM bit
intrinsics to LLVM dialect. These intrinsics help with SPIR-V to
LLVM conversion, allowing a direct mapping from `spv.BitReverse`
and `spv.BitCount` respectively. Tests are added to `roundtrip.mlir`
and `llvm-intrinsics.mlir`.

Differential Revision: https://reviews.llvm.org/D82285
The file was modifiedmlir/include/mlir/Dialect/LLVMIR/LLVMOps.td
The file was modifiedmlir/test/Dialect/LLVMIR/roundtrip.mlir
The file was modifiedmlir/test/Target/llvmir-intrinsics.mlir
Commit 0a9e7d0b6befad866dfd61f05b774247e0867121 by waltermelon
[vscode] set default values for terminateDebuggee for the disconnect request

Summary:
Recently I've noticed that VSCode sometimes doesn't send the terminateDebuggee flag within the disconnectRequest,
even though lldb-vscode sets the terminateDebuggee capability correctly.
This has been causing that inferiors don't die after the debug session ends, and many users have reported issues because of this.

An easy way to mitigate this is to set better default values for the terminateDebuggee field in the disconnect request.
I'm assuming that for a launch request, the default will be true, and for attach it'll be false.

Reviewers: clayborg, labath, aadsm

Subscribers: lldb-commits

Tags: #lldb

Differential Revision: https://reviews.llvm.org/D81200
The file was addedlldb/test/API/tools/lldb-vscode/disconnect/Makefile
The file was modifiedlldb/tools/lldb-vscode/VSCode.h
The file was addedlldb/test/API/tools/lldb-vscode/disconnect/TestVSCode_disconnect.py
The file was modifiedlldb/tools/lldb-vscode/VSCode.cpp
The file was modifiedlldb/packages/Python/lldbsuite/test/tools/lldb-vscode/lldbvscode_testcase.py
The file was modifiedlldb/tools/lldb-vscode/lldb-vscode.cpp
The file was addedlldb/test/API/tools/lldb-vscode/disconnect/main.cpp
Commit e618ccbf431f6730edb6d1467a127c3a52fd57f7 by i
[ELF] Resolve relocations in .debug_* referencing (discarded symbols or ICF folded section symbols) to tombstone values

See D59553, https://lists.llvm.org/pipermail/llvm-dev/2020-May/141885.html and
https://sourceware.org/pipermail/binutils/2020-May/111357.html for
extensive discussions on a tombstone value.
See http://www.dwarfstd.org/ShowIssue.php?issue=200609.1
(Reserve an address value for "not present") for a DWARF enhancement proposal.

We resolve such relocations to a tombstone value to indicate that the address is invalid.
This solves several problems (the normal behavior is to resolve the relocation to the addend):

* For an empty function in a collected section, a pair of (0,0) can
  terminate .debug_loc and .debug_ranges (as of binutils 2.34, GNU ld
  resolves such a relocation to 1 to avoid the .debug_ranges issue)
* If DW_AT_high_pc is sufficiently large, the address range can collide
  with a regular code range of low address (https://bugs.llvm.org/show_bug.cgi?id=41124 )
* If a text section is folded into another by ICF, we may leave entries
  in multiple CUs claiming ownership of the same range of code, which can
  confuse consumers.
* Debug information associated with COMDAT sections can have problems
  similar to ICF, but is more complex - thus not addressed by this patch.

For pre-DWARF-v5 .debug_loc and .debug_ranges, a pair of 0 can terminate
entries (invalidating subsequent ranges).
-1 is a reserved value with special meaning (base address selection entry) which can't be used either.
Use -2 instead.

For all other .debug_*, use UINT32_MAX for 32-bit targets and UINT64_MAX
for 64-bit targets. In the code, we intentionally use
`uint64_t tombstone = UINT64_MAX` for 32-bit targets as well: this matches
SignExtend64 as used in `relocateAlloc`. (Actually UINT32_MAX does not work for R_386_32)

Note 0, we only special case `target->symbolicRel` (R_X86_64_64, R_AARCH64_ABS64, R_PPC64_ADDR64), not
short-range absolute relocations (e.g. R_X86_64_32). Only forms like DW_FORM_addr need to be special cased.
They can hold an arbitrary address (must be 64-bit on a 64-bit target). (In theory,
producers can make use of small code model to emit 32-bit relocations. This doesn't seem to be leveraged.)

Note 1, we have to ignore the addend, because we don't want to resolve
DW_AT_low_pc (which may have a non-zero addend) to -1+addend (wrap
around to a low address):

  __attribute__((section(".text.x"))) void f1() { }
  __attribute__((section(".text.x"))) void f2() { } // DW_AT_low_pc has a non-zero addend

Note 2, if the prevailing copy does not have debugging information while
a non-prevailing copy has (partial debug build), we don't do extra work
to attach debugging information to the prevailing definition.  (clang
has a lot of debug info optimizations that are on-by-default that assume
the whole program is built with debug info).

  clang -c -ffunction-sections a.cc    # prevailing copy has no debug info
  clang -c -ffunction-sections -g b.cc

Reviewed By: dblaikie, avl, jhenderson

Differential Revision: https://reviews.llvm.org/D81784
The file was addedlld/test/ELF/debug-dead-reloc.s
The file was addedlld/test/ELF/debug-dead-reloc-icf.s
The file was modifiedlld/ELF/InputSection.cpp
The file was addedlld/test/ELF/debug-dead-reloc-32.s
Commit fb34345e363ae00e6a2f7979ef524f4a4ffae582 by mikhail.ramalho
Change CMake so that we only look for Z3 when LLVM_ENABLE_Z3_SOLVER is enabled

Reviewers: mikhail.ramalho

Reviewed By: mikhail.ramalho

Subscribers: mehdi_amini, mgorny, mikhail.ramalho, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D75544
The file was modifiedllvm/CMakeLists.txt
Commit cc9d69385659be32178506a38b4f2e112ed01ad4 by mahesha.comp
[AMDGPU/MemOpsCluster] Implement new heuristic for computing max mem ops cluster size

Summary:
Make use of both the - (1) clustered bytes and (2) cluster length, to decide on
the max number of mem ops that can be clustered. On an average, when loads
are dword or smaller, consider `5` as max threshold, otherwise `4`. This
heuristic is purely based on different experimentation conducted, and there is
no analytical logic here.

Reviewers: foad, rampitec, arsenm, vpykhtin

Reviewed By: rampitec

Subscribers: llvm-commits, kerbowa, hiraditya, t-tye, Anastasia, tpr, dstuttard, yaxunl, nhaehnle, wdng, jvesely, kzhuravl, thakis

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D82393
The file was modifiedllvm/lib/Target/AMDGPU/SIInstrInfo.cpp
The file was modifiedllvm/test/CodeGen/AMDGPU/GlobalISel/llvm.amdgcn.div.fmas.ll
The file was modifiedllvm/test/CodeGen/AMDGPU/promote-constOffset-to-imm.ll
The file was modifiedllvm/test/CodeGen/AMDGPU/trunc-store-i64.ll
The file was modifiedllvm/test/CodeGen/AMDGPU/amdhsa-trap-num-sgprs.ll
The file was modifiedllvm/test/CodeGen/AMDGPU/sgpr-control-flow.ll
The file was modifiedllvm/test/CodeGen/AMDGPU/store-weird-sizes.ll
The file was modifiedllvm/test/CodeGen/AMDGPU/memory_clause.ll
The file was modifiedllvm/test/CodeGen/AMDGPU/shift-i128.ll
The file was modifiedllvm/test/CodeGen/AMDGPU/global-saddr.ll
The file was modifiedllvm/test/CodeGen/AMDGPU/salu-to-valu.ll
The file was modifiedllvm/test/CodeGen/AMDGPU/udivrem.ll
The file was modifiedllvm/test/CodeGen/AMDGPU/kernel-args.ll
The file was modifiedllvm/test/CodeGen/AMDGPU/insert_vector_elt.ll
Commit 1db1a08ddae6174e0e0bf6f8a8404cef9091b68b by jean-michel.gorius
[mlir] Avoid pontentially ambiguous class name

Summary: The Pass class exists in both the mlir and the llvm namespaces. Use the fully qualified class name to avoid any ambiguities.

Reviewers: rriddle

Reviewed By: rriddle

Subscribers: mehdi_amini, jpienaar, shauheen, antiagainst, nicolasvasilache, arpith-jacob, mgester, lucyrfox, aartbik, liufengdb, stephenneuendorffer, Joonsoo, grosul1, jurahul, msifontes

Tags: #mlir

Differential Revision: https://reviews.llvm.org/D82371
The file was modifiedmlir/tools/mlir-tblgen/PassGen.cpp
Commit 3c79212319d878b07ef259d735b52b379f774e25 by ditaliano
[Host] Check for TARGET_OS_EMBEDDED instead of listing architectures.

With the advent of Apple Silicon, checking for the architectures
specifically is not correct anymore. This code is only supposed to
run on embedded devices (iPhones et similia), so mark it accordingly.
The file was modifiedlldb/source/Host/macosx/objcxx/HostInfoMacOSX.mm
Commit 63d597093cccbb8d4962cf490e2d754a73a77e64 by ditaliano
[ObjectFileMachO] Check for TARGET_EMBEDDED instead of listing architectures.

Now that Apple Silicon is a thing, we need to generalize the check.
The file was modifiedlldb/source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp
Commit 33ece57241d8ad46cb91eca483f05515849a85e5 by ditaliano
Generalize TestFormattersBoolRefPtr to work on Apple Silicon.
The file was modifiedlldb/test/API/functionalities/data-formatter/boolreference/TestFormattersBoolRefPtr.py
Commit 74ab1da0285fb1f37fdb4648e2c677e97a2a5231 by waltermelon
Redo of Add terminateCommands to lldb-vscode protocol

Summary:
This redoes https://reviews.llvm.org/D79726 and fixes two things.
- The logic that determines whether to automatically disconnect during the tear down is not very dumb compared to the original implementation. Each test will determine whether to do that or not.
- The terminate commands and terminate event were being sent after the disconnect response was sent to the IDE. That was not good, as VSCode stops the debug session as soon as it receives a disconnect response. Now, the terminate event and terminateEvents are being executed before the disconnect response is sent. This ensures that any connection between the IDE and lldb-vscode is alive while the terminate commands are executed. Besides, it also allows displaying the output of the terminate commands on the debug console, as it's still alive.

Reviewers: clayborg, aadsm, kusmour, labath

Subscribers: lldb-commits

Tags: #lldb

Differential Revision: https://reviews.llvm.org/D81978
The file was modifiedlldb/tools/lldb-vscode/lldb-vscode.cpp
The file was modifiedlldb/test/API/tools/lldb-vscode/launch/TestVSCode_launch.py
The file was modifiedlldb/tools/lldb-vscode/VSCode.cpp
The file was modifiedlldb/packages/Python/lldbsuite/test/tools/lldb-vscode/lldbvscode_testcase.py
The file was modifiedlldb/tools/lldb-vscode/VSCode.h
The file was modifiedlldb/test/API/tools/lldb-vscode/attach/TestVSCode_attach.py
The file was modifiedlldb/packages/Python/lldbsuite/test/tools/lldb-vscode/vscode.py
The file was modifiedlldb/tools/lldb-vscode/README.md
Commit 3a55a2a97fd419c1b6c5299b3523846a9fa9bc52 by kamau.bridgeman
[LLD][PowerPC] Add support for R_PPC64_PCREL34

Add support for the 34bit relocation R_PPC64_PCREL34 for PC Relative in LLD.
The file was modifiedlld/ELF/Arch/PPC64.cpp
The file was addedlld/test/ELF/ppc64-reloc-pcrel34-overflow.s
The file was addedlld/test/ELF/ppc64-reloc-pcrel34.s
Commit ff4de8683ad1802dbf20d0286861bd98462e92e2 by flo
[DSE,MSSA] Treat `store 0` after calloc as noop stores.

This patch extends storeIsNoop to also detect stores of 0 to an calloced
object. This basically ports the logic from legacy DSE to the MemorySSA
backed version.

It triggers in a few cases on MultiSource, SPEC2000, SPEC2006 with -O3
LTO:

Same hash: 218 (filtered out)
Remaining: 19
Metric: dse.NumNoopStores

Program                                        base   patch2 diff
test-suite...CFP2000/177.mesa/177.mesa.test     1.00  15.00 1400.0%
test-suite...6/482.sphinx3/482.sphinx3.test     1.00  14.00 1300.0%
test-suite...lications/ClamAV/clamscan.test     2.00  28.00 1300.0%
test-suite...CFP2006/433.milc/433.milc.test     1.00   8.00 700.0%
test-suite...pplications/oggenc/oggenc.test     2.00   9.00 350.0%
test-suite.../CINT2000/176.gcc/176.gcc.test     6.00   6.00  0.0%
test-suite.../CINT2006/403.gcc/403.gcc.test    NaN   137.00  nan%
test-suite...libquantum/462.libquantum.test    NaN     3.00  nan%
test-suite...6/464.h264ref/464.h264ref.test    NaN     7.00  nan%
test-suite...decode/alacconvert-decode.test    NaN     2.00  nan%
test-suite...encode/alacconvert-encode.test    NaN     2.00  nan%
test-suite...ications/JM/ldecod/ldecod.test    NaN     9.00  nan%
test-suite...ications/JM/lencod/lencod.test    NaN    39.00  nan%
test-suite.../Applications/lemon/lemon.test    NaN     2.00  nan%
test-suite...pplications/treecc/treecc.test    NaN     4.00  nan%
test-suite...hmarks/McCat/08-main/main.test    NaN     4.00  nan%
test-suite...nsumer-lame/consumer-lame.test    NaN     3.00  nan%
test-suite.../Prolangs-C/bison/mybison.test    NaN     1.00  nan%
test-suite...arks/mafft/pairlocalalign.test    NaN    30.00  nan%

Reviewers: efriedma, zoecarver, asbirlea

Reviewed By: asbirlea

Differential Revision: https://reviews.llvm.org/D82204
The file was modifiedllvm/test/Transforms/DeadStoreElimination/MSSA/simple.ll
The file was modifiedllvm/test/Transforms/DeadStoreElimination/MSSA/calloc-store.ll
The file was modifiedllvm/lib/Transforms/Scalar/DeadStoreElimination.cpp
The file was modifiedllvm/test/Transforms/DeadStoreElimination/MSSA/simple-todo.ll
Commit dc72be4e0130675833c56d4d72e22a56872876ea by sivachandra
[libc][Obvious] Fix few typos in tests.
The file was modifiedlibc/test/src/math/floorf_test.cpp
The file was modifiedlibc/test/src/math/ceilf_test.cpp
The file was modifiedlibc/test/src/math/modff_test.cpp
The file was modifiedlibc/test/src/math/truncf_test.cpp
The file was modifiedlibc/test/src/math/roundf_test.cpp
The file was modifiedlibc/test/src/math/frexpf_test.cpp
Commit 52e86797ba687d89a2e9a85c135b7145a5f14739 by nikita.ppv
[IR] Remove unnecessary uint64_t casts (NFC)

As pointed out by foad, it's not necessary to work on uint64_t
here. The values used here fit uint8_t.
The file was modifiedllvm/lib/IR/AttributeImpl.h