[PATCH] D142444: [IPSCCP][FuncSpec] Fix compiler crash 60191.

[Nikita Popov via Phabricator via llvm-commits]

nikic added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/FunctionSpecialization.cpp:627
+    if (CS->arg_size() != CalledFunction->arg_size())
+      continue;
 
----------------
Is it okay if the arguments have different types? (If no, then you'll want to compare getFunctionType()).


================
Comment at: llvm/test/Transforms/FunctionSpecialization/compiler-crash-60191.ll:1
+; RUN: opt -S -mtriple=amdgcn-amd-amdhsa --passes="default<O3>" < %s
+
----------------
Can you avoid the triple in the test?

Also, is it possible to reduce the test such that it only requires running ipsccp, and not the whole opt pipeline?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142444/new/

https://reviews.llvm.org/D142444