[PATCH] D101610: [AMDGPU] Tidy up some simple expressions for clarity NFC

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 30 02:59:55 PDT 2021


foad accepted this revision.
foad added a comment.
This revision is now accepted and ready to land.

Looks obviously fine.



================
Comment at: llvm/lib/Target/AMDGPU/SIInstrInfo.cpp:29
 #include "llvm/Support/CommandLine.h"
+#include "llvm/Support/MathExtras.h"
 #include "llvm/Target/TargetMachine.h"
----------------
Do you need this? I thought it was already included by default (not sure exactly how, though).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101610/new/

https://reviews.llvm.org/D101610



More information about the llvm-commits mailing list