[llvm] [SelectionDAG] Move SelectionDAG::getAllOnesConstant out of line. NFC (PR #102995)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 13 01:06:42 PDT 2024


================
@@ -675,10 +675,7 @@ class SelectionDAG {
                       bool isTarget = false, bool isOpaque = false);
 
   SDValue getAllOnesConstant(const SDLoc &DL, EVT VT, bool IsTarget = false,
-                             bool IsOpaque = false) {
-    return getConstant(APInt::getAllOnes(VT.getScalarSizeInBits()), DL, VT,
----------------
arsenm wrote:

But that sounds like a bug, generally constants are sign extended in other contexts in llvm (e.g. in MachineOperand) 

https://github.com/llvm/llvm-project/pull/102995


More information about the llvm-commits mailing list