[PATCH] D142944: [Doc] Removes extra argument of ReplaceInstWithValue()

Vasileios Porpodas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 30 17:35:28 PST 2023


vporpo added inline comments.


================
Comment at: llvm/docs/ProgrammersManual.rst:2959-2960
 
-    ReplaceInstWithValue(instToReplace->getParent(), ii,
+    ReplaceInstWithValue(ii,
                          Constant::getNullValue(PointerType::getUnqual(Type::Int32Ty)));
 
----------------
dbabokin wrote:
> Should the arguments go on the same line now?
It is already > 80 columns wide, so yeah why not.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142944



More information about the llvm-commits mailing list