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

Wed Jan 22 06:26:14 PST 2025

llvm-ci wrote:

LLVM Buildbot has detected a new failure on builder `llvm-clang-aarch64-darwin` running on `doug-worker-5` while building `llvm` at step 5 "build-unified-tree".

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

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

```
Step 5 (build-unified-tree) failure: build (failure)
...
99.106 [3/8/5989] Building CXX object third-party/benchmark/src/CMakeFiles/benchmark.dir/statistics.cc.o
99.114 [3/7/5990] Building CXX object third-party/benchmark/src/CMakeFiles/benchmark.dir/sysinfo.cc.o
99.114 [3/6/5991] Building CXX object third-party/benchmark/src/CMakeFiles/benchmark.dir/string_util.cc.o
99.117 [3/5/5992] Building CXX object third-party/benchmark/src/CMakeFiles/benchmark.dir/timers.cc.o
99.119 [2/5/5993] Copying llvm-locstats into /Users/buildbot/buildbot-root/aarch64-darwin/build/./bin
99.120 [2/4/5994] Building CXX object third-party/benchmark/src/CMakeFiles/benchmark_main.dir/benchmark_main.cc.o
99.125 [2/3/5995] Linking CXX static library lib/libbenchmark.a
99.130 [1/3/5996] Linking CXX static library lib/libbenchmark_main.a
99.237 [1/2/5997] Linking CXX executable unittests/tools/llvm-exegesis/LLVMExegesisTests
100.280 [1/1/5998] Building CXX object unittests/CodeGen/GlobalISel/CMakeFiles/GlobalISelTests.dir/CSETest.cpp.o
FAILED: unittests/CodeGen/GlobalISel/CMakeFiles/GlobalISelTests.dir/CSETest.cpp.o 
CCACHE_CPP2=yes CCACHE_HASHDIR=yes /opt/homebrew/bin/ccache /usr/bin/clang++ -DGTEST_HAS_RTTI=0 -DLLVM_BUILD_STATIC -D_DEBUG -D_GLIBCXX_ASSERTIONS -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -I/Users/buildbot/buildbot-root/aarch64-darwin/build/unittests/CodeGen/GlobalISel -I/Users/buildbot/buildbot-root/aarch64-darwin/llvm-project/llvm/unittests/CodeGen/GlobalISel -I/Users/buildbot/buildbot-root/aarch64-darwin/build/include -I/Users/buildbot/buildbot-root/aarch64-darwin/llvm-project/llvm/include -I/Users/buildbot/buildbot-root/aarch64-darwin/llvm-project/third-party/unittest/googletest/include -I/Users/buildbot/buildbot-root/aarch64-darwin/llvm-project/third-party/unittest/googlemock/include -isystem /opt/homebrew/include -fPIC -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 -O3 -DNDEBUG -std=c++17 -arch arm64 -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX14.4.sdk  -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 /Users/buildbot/buildbot-root/aarch64-darwin/llvm-project/llvm/unittests/CodeGen/GlobalISel/CSETest.cpp
/Users/buildbot/buildbot-root/aarch64-darwin/llvm-project/llvm/unittests/CodeGen/GlobalISel/CSETest.cpp:84:14: error: no member named 'setFlag' in 'llvm::MachineInstrBuilder'; did you mean 'setMIFlag'?
    MIBFAdd2.setFlag(MachineInstr::FmNsz);
             ^~~~~~~
             setMIFlag
/Users/buildbot/buildbot-root/aarch64-darwin/llvm-project/llvm/include/llvm/CodeGen/MachineInstrBuilder.h:280:30: note: 'setMIFlag' declared here
  const MachineInstrBuilder &setMIFlag(MachineInstr::MIFlag Flag) const {
                             ^
/Users/buildbot/buildbot-root/aarch64-darwin/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 '.'?
    MIBFAdd2.clearFlag(MachineInstr::FmNsz);
            ^
            ->
/Users/buildbot/buildbot-root/aarch64-darwin/llvm-project/llvm/unittests/CodeGen/GlobalISel/CSETest.cpp:97:14: error: no member named 'setFlag' in 'llvm::MachineInstrBuilder'; did you mean 'setMIFlag'?
    MIBFSub2.setFlag(MachineInstr::FmNoNans);
             ^~~~~~~
             setMIFlag
/Users/buildbot/buildbot-root/aarch64-darwin/llvm-project/llvm/include/llvm/CodeGen/MachineInstrBuilder.h:280:30: note: 'setMIFlag' declared here
  const MachineInstrBuilder &setMIFlag(MachineInstr::MIFlag Flag) const {
                             ^
/Users/buildbot/buildbot-root/aarch64-darwin/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 '.'?
    MIBFSub2.clearFlag(MachineInstr::FmNoNans);
            ^
            ->
/Users/buildbot/buildbot-root/aarch64-darwin/llvm-project/llvm/unittests/CodeGen/GlobalISel/CSETest.cpp:110:14: error: no member named 'setFlag' in 'llvm::MachineInstrBuilder'; did you mean 'setMIFlag'?
    MIBFMul2.setFlag(MachineInstr::FmNoNans);
             ^~~~~~~
             setMIFlag
/Users/buildbot/buildbot-root/aarch64-darwin/llvm-project/llvm/include/llvm/CodeGen/MachineInstrBuilder.h:280:30: note: 'setMIFlag' declared here
  const MachineInstrBuilder &setMIFlag(MachineInstr::MIFlag Flag) const {
                             ^
/Users/buildbot/buildbot-root/aarch64-darwin/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 '.'?
    MIBFMul2.clearFlag(MachineInstr::FmNoNans);
            ^
            ->
/Users/buildbot/buildbot-root/aarch64-darwin/llvm-project/llvm/unittests/CodeGen/GlobalISel/CSETest.cpp:123:14: error: no member named 'setFlag' in 'llvm::MachineInstrBuilder'; did you mean 'setMIFlag'?
    MIBFDiv2.setFlag(MachineInstr::FmNoNans);
             ^~~~~~~
             setMIFlag

```

</details>

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