[PATCH] D113672: [AMDGPU][NFC] Fix typos

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 11 06:43:58 PST 2021


foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.h:112
   void legalizeUnsignedDIV_REM64Impl(MachineIRBuilder &B, Register DstDivReg,
-                                     Register DstRemReg, Register Numer,
+                                     Register DstRemReg, Register Num,
                                      Register Denom) const;
----------------
If you're changing this, maybe use "Den" too like in the function above?


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPULibFunc.cpp:780
+// DvNN_ type is GCC extension for vectors and is a subject for the
+// substitution.
 
----------------
I can't see a typo fix here. Is it just line wrapping?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113672



More information about the llvm-commits mailing list