github-actions[bot] wrote: <!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning: <details> <summary> You can test this locally with the following command: </summary> ``````````bash git-clang-format --diff f7788618dd24e5366709a70c3e16cd5a0ca0a173 23fec587e7ba2c6bb34aa0a469bc17f9d4cfbb87 --extensions c,cpp -- clang/lib/Basic/Targets/WebAssembly.cpp clang/test/Preprocessor/wasm-target-features.c llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp llvm/lib/Target/WebAssembly/WebAssemblySelectionDAGInfo.cpp `````````` </details> <details> <summary> View the diff from clang-format here. </summary> ``````````diff diff --git a/llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp b/llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp index e7e84ed6b5..65c8947f74 100644 --- a/llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp +++ b/llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp @@ -565,8 +565,7 @@ static MachineBasicBlock *LowerFPToInt(MachineInstr &MI, DebugLoc DL, // instuction to handle the zero-length case. static MachineBasicBlock *LowerMemcpy(MachineInstr &MI, DebugLoc DL, MachineBasicBlock *BB, - const TargetInstrInfo &TII, - bool Int64) { + const TargetInstrInfo &TII, bool Int64) { MachineRegisterInfo &MRI = BB->getParent()->getRegInfo(); MachineOperand DstMem = MI.getOperand(0); @@ -585,7 +584,8 @@ static MachineBasicBlock *LowerMemcpy(MachineInstr &MI, DebugLoc DL, MachineBasicBlock *TrueMBB = F->CreateMachineBasicBlock(LLVMBB); MachineBasicBlock *DoneMBB = F->CreateMachineBasicBlock(LLVMBB); - unsigned MemoryCopy = Int64 ? WebAssembly::MEMORY_COPY_A64 : WebAssembly::MEMORY_COPY_A32; + unsigned MemoryCopy = + Int64 ? WebAssembly::MEMORY_COPY_A64 : WebAssembly::MEMORY_COPY_A32; MachineFunction::iterator It = ++BB->getIterator(); F->insert(It, TrueMBB); @@ -624,8 +624,7 @@ static MachineBasicBlock *LowerMemcpy(MachineInstr &MI, DebugLoc DL, // instuction to handle the zero-length case. static MachineBasicBlock *LowerMemset(MachineInstr &MI, DebugLoc DL, MachineBasicBlock *BB, - const TargetInstrInfo &TII, - bool Int64) { + const TargetInstrInfo &TII, bool Int64) { MachineRegisterInfo &MRI = BB->getParent()->getRegInfo(); MachineOperand Mem = MI.getOperand(0); @@ -643,7 +642,8 @@ static MachineBasicBlock *LowerMemset(MachineInstr &MI, DebugLoc DL, MachineBasicBlock *TrueMBB = F->CreateMachineBasicBlock(LLVMBB); MachineBasicBlock *DoneMBB = F->CreateMachineBasicBlock(LLVMBB); - unsigned MemoryFill = Int64 ? WebAssembly::MEMORY_FILL_A64 : WebAssembly::MEMORY_FILL_A32; + unsigned MemoryFill = + Int64 ? WebAssembly::MEMORY_FILL_A64 : WebAssembly::MEMORY_FILL_A32; MachineFunction::iterator It = ++BB->getIterator(); F->insert(It, TrueMBB); @@ -664,11 +664,7 @@ static MachineBasicBlock *LowerMemset(MachineInstr &MI, DebugLoc DL, BuildMI(BB, DL, TII.get(Eqz), EqzReg).add(NoKillLen); - BuildMI(TrueMBB, DL, TII.get(MemoryFill)) - .add(Mem) - .add(Dst) - .add(Val) - .add(Len); + BuildMI(TrueMBB, DL, TII.get(MemoryFill)).add(Mem).add(Dst).add(Val).add(Len); // Create the CFG triangle. BuildMI(BB, DL, TII.get(WebAssembly::BR_IF)).addMBB(DoneMBB).addReg(EqzReg); diff --git a/llvm/lib/Target/WebAssembly/WebAssemblySelectionDAGInfo.cpp b/llvm/lib/Target/WebAssembly/WebAssemblySelectionDAGInfo.cpp index 48119de121..d51bfeb6d8 100644 --- a/llvm/lib/Target/WebAssembly/WebAssemblySelectionDAGInfo.cpp +++ b/llvm/lib/Target/WebAssembly/WebAssemblySelectionDAGInfo.cpp @@ -32,9 +32,9 @@ SDValue WebAssemblySelectionDAGInfo::EmitTargetCodeForMemcpy( // Use `MEMCPY` here instead of `MEMORY_COPY` because `memory.copy` traps // if the pointers are invalid even if the length is zero. `MEMCPY` gets // extra code to handle this in the way that LLVM IR expects. - return DAG.getNode(WebAssemblyISD::MEMCPY, DL, MVT::Other, - {Chain, MemIdx, MemIdx, Dst, Src, - DAG.getZExtOrTrunc(Size, DL, LenMVT)}); + return DAG.getNode( + WebAssemblyISD::MEMCPY, DL, MVT::Other, + {Chain, MemIdx, MemIdx, Dst, Src, DAG.getZExtOrTrunc(Size, DL, LenMVT)}); } SDValue WebAssemblySelectionDAGInfo::EmitTargetCodeForMemmove( @@ -62,7 +62,7 @@ SDValue WebAssemblySelectionDAGInfo::EmitTargetCodeForMemset( // extra code to handle this in the way that LLVM IR expects. // // Only low byte matters for val argument, so anyext the i8 - return DAG.getNode(WebAssemblyISD::MEMSET, DL, MVT::Other, Chain, MemIdx, - Dst, DAG.getAnyExtOrTrunc(Val, DL, MVT::i32), + return DAG.getNode(WebAssemblyISD::MEMSET, DL, MVT::Other, Chain, MemIdx, Dst, + DAG.getAnyExtOrTrunc(Val, DL, MVT::i32), DAG.getZExtOrTrunc(Size, DL, LenMVT)); } `````````` </details> https://github.com/llvm/llvm-project/pull/112049 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits