[llvm] [GISel] Add more FP opcodes to CSE (PR #123624)

Wed Jan 22 06:27:53 PST 2025

llvm-ci wrote:

LLVM Buildbot has detected a new failure on builder `clang-debian-cpp20` running on `clang-debian-cpp20` while building `llvm` at step 6 "test-build-unified-tree-check-all".

Full details are available at: https://lab.llvm.org/buildbot/#/builders/108/builds/8427

<details>
<summary>Here is the relevant piece of the build log for the reference</summary>

```
Step 6 (test-build-unified-tree-check-all) failure: test (failure)
...
36.970 [2/15/1190] Linking CXX executable unittests/tools/llvm-profgen/LLVMProfgenTests
37.069 [2/14/1191] Linking CXX executable unittests/Transforms/Vectorize/SandboxVectorizer/SandboxVectorizerTests
37.106 [2/13/1192] Linking CXX executable unittests/Transforms/Vectorize/VectorizeTests
37.156 [2/12/1193] Linking CXX executable unittests/tools/llvm-profdata/LLVMProfdataTests
37.334 [2/11/1194] Linking CXX executable unittests/Transforms/IPO/IPOTests
37.686 [2/10/1195] Linking CXX executable unittests/Target/X86/X86Tests
37.741 [2/9/1196] Linking CXX executable unittests/Transforms/Instrumentation/InstrumentationTests
37.777 [2/8/1197] Linking CXX executable unittests/tools/llvm-cfi-verify/CFIVerifyTests
37.860 [2/7/1198] Linking CXX executable unittests/Transforms/Coroutines/CoroTests
38.079 [2/6/1199] Building CXX object unittests/CodeGen/GlobalISel/CMakeFiles/GlobalISelTests.dir/CSETest.cpp.o
FAILED: unittests/CodeGen/GlobalISel/CMakeFiles/GlobalISelTests.dir/CSETest.cpp.o 
ccache /usr/bin/clang++-17 -DGTEST_HAS_RTTI=0 -DLLVM_BUILD_STATIC -D_DEBUG -D_GLIBCXX_ASSERTIONS -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -I/vol/worker/clang-debian-cpp20/clang-debian-cpp20/build/unittests/CodeGen/GlobalISel -I/vol/worker/clang-debian-cpp20/clang-debian-cpp20/llvm-project/llvm/unittests/CodeGen/GlobalISel -I/vol/worker/clang-debian-cpp20/clang-debian-cpp20/build/include -I/vol/worker/clang-debian-cpp20/clang-debian-cpp20/llvm-project/llvm/include -I/vol/worker/clang-debian-cpp20/clang-debian-cpp20/llvm-project/third-party/unittest/googletest/include -I/vol/worker/clang-debian-cpp20/clang-debian-cpp20/llvm-project/third-party/unittest/googlemock/include -Wno-deprecated-enum-enum-conversion -Wno-deprecated-declarations -Wno-deprecated-anon-enum-enum-conversion -Wno-ambiguous-reversed-operator -fPIC -fno-semantic-interposition -fvisibility-inlines-hidden -Werror=date-time -Werror=unguarded-availability-new -Wall -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wmissing-field-initializers -pedantic -Wno-long-long -Wc++98-compat-extra-semi -Wimplicit-fallthrough -Wcovered-switch-default -Wno-noexcept-type -Wnon-virtual-dtor -Wdelete-non-virtual-dtor -Wsuggest-override -Wstring-conversion -Wmisleading-indentation -Wctad-maybe-unsupported -fdiagnostics-color -ffunction-sections -fdata-sections -O3 -DNDEBUG -std=c++20  -Wno-variadic-macros -Wno-gnu-zero-variadic-macro-arguments -fno-exceptions -funwind-tables -fno-rtti -UNDEBUG -Wno-suggest-override -MD -MT unittests/CodeGen/GlobalISel/CMakeFiles/GlobalISelTests.dir/CSETest.cpp.o -MF unittests/CodeGen/GlobalISel/CMakeFiles/GlobalISelTests.dir/CSETest.cpp.o.d -o unittests/CodeGen/GlobalISel/CMakeFiles/GlobalISelTests.dir/CSETest.cpp.o -c /vol/worker/clang-debian-cpp20/clang-debian-cpp20/llvm-project/llvm/unittests/CodeGen/GlobalISel/CSETest.cpp
/vol/worker/clang-debian-cpp20/clang-debian-cpp20/llvm-project/llvm/unittests/CodeGen/GlobalISel/CSETest.cpp:84:14: error: no member named 'setFlag' in 'llvm::MachineInstrBuilder'; did you mean 'setMIFlag'?
   84 |     MIBFAdd2.setFlag(MachineInstr::FmNsz);
      |              ^~~~~~~
      |              setMIFlag
/vol/worker/clang-debian-cpp20/clang-debian-cpp20/llvm-project/llvm/include/llvm/CodeGen/MachineInstrBuilder.h:280:30: note: 'setMIFlag' declared here
  280 |   const MachineInstrBuilder &setMIFlag(MachineInstr::MIFlag Flag) const {
      |                              ^
/vol/worker/clang-debian-cpp20/clang-debian-cpp20/llvm-project/llvm/unittests/CodeGen/GlobalISel/CSETest.cpp:87:13: error: no member named 'clearFlag' in 'llvm::MachineInstrBuilder'; did you mean to use '->' instead of '.'?
   87 |     MIBFAdd2.clearFlag(MachineInstr::FmNsz);
      |             ^
      |             ->
/vol/worker/clang-debian-cpp20/clang-debian-cpp20/llvm-project/llvm/unittests/CodeGen/GlobalISel/CSETest.cpp:97:14: error: no member named 'setFlag' in 'llvm::MachineInstrBuilder'; did you mean 'setMIFlag'?
   97 |     MIBFSub2.setFlag(MachineInstr::FmNoNans);
      |              ^~~~~~~
      |              setMIFlag
/vol/worker/clang-debian-cpp20/clang-debian-cpp20/llvm-project/llvm/include/llvm/CodeGen/MachineInstrBuilder.h:280:30: note: 'setMIFlag' declared here
  280 |   const MachineInstrBuilder &setMIFlag(MachineInstr::MIFlag Flag) const {
      |                              ^
/vol/worker/clang-debian-cpp20/clang-debian-cpp20/llvm-project/llvm/unittests/CodeGen/GlobalISel/CSETest.cpp:100:13: error: no member named 'clearFlag' in 'llvm::MachineInstrBuilder'; did you mean to use '->' instead of '.'?
  100 |     MIBFSub2.clearFlag(MachineInstr::FmNoNans);
      |             ^
      |             ->
/vol/worker/clang-debian-cpp20/clang-debian-cpp20/llvm-project/llvm/unittests/CodeGen/GlobalISel/CSETest.cpp:110:14: error: no member named 'setFlag' in 'llvm::MachineInstrBuilder'; did you mean 'setMIFlag'?
  110 |     MIBFMul2.setFlag(MachineInstr::FmNoNans);
      |              ^~~~~~~
      |              setMIFlag
/vol/worker/clang-debian-cpp20/clang-debian-cpp20/llvm-project/llvm/include/llvm/CodeGen/MachineInstrBuilder.h:280:30: note: 'setMIFlag' declared here
  280 |   const MachineInstrBuilder &setMIFlag(MachineInstr::MIFlag Flag) const {
      |                              ^
/vol/worker/clang-debian-cpp20/clang-debian-cpp20/llvm-project/llvm/unittests/CodeGen/GlobalISel/CSETest.cpp:113:13: error: no member named 'clearFlag' in 'llvm::MachineInstrBuilder'; did you mean to use '->' instead of '.'?
  113 |     MIBFMul2.clearFlag(MachineInstr::FmNoNans);
      |             ^
      |             ->
/vol/worker/clang-debian-cpp20/clang-debian-cpp20/llvm-project/llvm/unittests/CodeGen/GlobalISel/CSETest.cpp:123:14: error: no member named 'setFlag' in 'llvm::MachineInstrBuilder'; did you mean 'setMIFlag'?
  123 |     MIBFDiv2.setFlag(MachineInstr::FmNoNans);
      |              ^~~~~~~
      |              setMIFlag

```

</details>

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