diff options
| author | Pierre van Houtryve <pierre.vanhoutryve@amd.com> | 2025-10-01 10:51:00 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-10-01 10:51:00 +0200 |
| commit | 14fcd81861aa1576d204b9146345c4426d81fc49 (patch) | |
| tree | 347d6509028958546f710645129a828772c2e32f /clang-tools-extra/clang-tidy/linuxkernel/LinuxKernelTidyModule.cpp | |
| parent | f80e7e139e3e677638233037499f8cba50c66b9e (diff) | |
| download | llvm-14fcd81861aa1576d204b9146345c4426d81fc49.zip llvm-14fcd81861aa1576d204b9146345c4426d81fc49.tar.gz llvm-14fcd81861aa1576d204b9146345c4426d81fc49.tar.bz2 | |
[AMDGPU][InsertWaitCnts] Refactor some helper functions, NFC (#161160)
- Remove one-line wrappers around a simple function call when they're
only used once or twice.
- Move very generic helpers into SIInstrInfo
- Delete unused functions
The goal is simply to reduce the noise in SIInsertWaitCnts without
hiding functionality. I focused on moving trivial helpers, or helpers
with very descriptive/verbose names (so it doesn't hide too much logic
away from the pass), and that have some reusability potential.
I'm also trying to make the code style more consistent. It doesn't make
sense to see a function call `TII->isXXX` then suddenly call a random
`isY` method that just wraps around `TII->isY`.
The context of this work is that I'm trying to learn how this pass
works, and while going through the code I noticed some little things
here and there that I thought would be good to fix.
Diffstat (limited to 'clang-tools-extra/clang-tidy/linuxkernel/LinuxKernelTidyModule.cpp')
0 files changed, 0 insertions, 0 deletions