[all-commits] [llvm/llvm-project] 8ed527: Reapply [TLI] Fix replace-with-veclib crash with i...

Paschalis Mpeis via All-commits all-commits at lists.llvm.org
Fri Jan 12 09:13:38 PST 2024


  Branch: refs/heads/users/paschalis-mpeis/reapply-fix-replace-veclib-crash
  Home:   https://github.com/llvm/llvm-project
  Commit: 8ed527c2a5dc9dd1b899caa5d1663e504b072069
      https://github.com/llvm/llvm-project/commit/8ed527c2a5dc9dd1b899caa5d1663e504b072069
  Author: Paschalis Mpeis <paschalis.mpeis at arm.com>
  Date:   2024-01-12 (Fri, 12 Jan 2024)

  Changed paths:
    M llvm/lib/CodeGen/ReplaceWithVeclib.cpp
    M llvm/unittests/Analysis/CMakeLists.txt
    A llvm/unittests/Analysis/ReplaceWithVecLibTest.cpp

  Log Message:
  -----------
  Reapply [TLI] Fix replace-with-veclib crash with invalid arguments (#77112)

Fix a crash of `replace-with-veclib` pass, when the arguments of the TLI
mapping do not match the original call.
Now, it simply ignores such cases.

Test require assertions as it accesses programmatically the debug log.

NOTE: Originally submitted by commit 9fdc568824b0, which was reverted by
a300b2403784, as it was causing some linking issues:
https://lab.llvm.org/buildbot/#/builders/234/builds/17734




More information about the All-commits mailing list