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 origin/main HEAD --extensions cpp,c,h --
clang/test/Driver/msp430-cpu.c clang/test/Preprocessor/msp430-defs.c
clang/lib/Basic/Targets/MSP430.cpp clang/lib/Basic/Targets/MSP430.h
clang/lib/Driver/ToolChains/CommonArgs.cpp clang/lib/Driver/ToolChains/Gnu.cpp
clang/lib/Driver/ToolChains/MSP430.cpp clang/lib/Driver/ToolChains/MSP430.h
clang/test/Driver/msp430-toolchain.c
llvm/lib/Target/MSP430/MSP430ISelDAGToDAG.cpp --diff_from_common_commit
``````````
:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/llvm/lib/Target/MSP430/MSP430ISelDAGToDAG.cpp
b/llvm/lib/Target/MSP430/MSP430ISelDAGToDAG.cpp
index de32bea4e..b341ac80a 100644
--- a/llvm/lib/Target/MSP430/MSP430ISelDAGToDAG.cpp
+++ b/llvm/lib/Target/MSP430/MSP430ISelDAGToDAG.cpp
@@ -372,8 +372,7 @@ bool MSP430DAGToDAGISel::tryIndexedStore(SDNode *N) {
StoreSDNode *ST = cast<StoreSDNode>(N);
SDValue Val = ST->getValue();
// Result 0 is the loaded value. Result 1 would be the incremented pointer.
- if (Val.getOpcode() != ISD::LOAD || Val.getResNo() != 0 ||
- !Val.hasOneUse())
+ if (Val.getOpcode() != ISD::LOAD || Val.getResNo() != 0 || !Val.hasOneUse())
return false;
LoadSDNode *LD = cast<LoadSDNode>(Val);
@@ -416,8 +415,9 @@ bool MSP430DAGToDAGISel::tryIndexedStore(SDNode *N) {
CurDAG->setNodeMemRefs(Mov, {LD->getMemOperand(), ST->getMemOperand()});
// The load results are (value, writeback, chain) and the store result is
- // (chain). Mov replaces both with the results of the operation: (writeback,
chain).
- // The loaded value needs no replacement because its only user was the store.
+ // (chain). Mov replaces both with the results of the operation: (writeback,
+ // chain). The loaded value needs no replacement because its only user was
the
+ // store.
ReplaceUses(SDValue(LD, 1), SDValue(Mov, 0));
ReplaceUses(SDValue(LD, 2), SDValue(Mov, 1));
ReplaceUses(SDValue(N, 0), SDValue(Mov, 1));
``````````
</details>
https://github.com/llvm/llvm-project/pull/235515
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
