[PATCH] D98884: [IR] Ignore bitcasts of function pointers which are only used as callees in callbase instruction

Fri Apr 9 13:11:42 PDT 2021

arsenm added a comment.

In D98884#2678593 <https://reviews.llvm.org/D98884#2678593>, @madhur13490 wrote:

> This change has wider impact than was expected as it affects several components and backends. Bootstrap builds fail with this change and as the buildbots show, this change generates a buggy Tablegen binary in release mode.
>
> Debugging:
>
> 1. Disabling the newly added code in hasAddressTaken(), we get sane Tablegen and builds pass. To note, debug builds pass with the change. This clearly shows that codegen is functionally different in release and debug mode for Tablegen.
> 2. Doing “ninja check-all” on stage1 build does not help although we have ~90000 tests.
> 3. Running valgrind also did not reveal any new things.
> 4. Doing ASAN build of the compiler led to several errors and drawing any conclusion from it is close to impossible for this patch.
>
> However, if I disable IPSCCP pass on top of this patch then everything turns green. I tried to fix trivial issues in the pass but it seems the pass needs deeper and major fixes to make this patch land.
>
> Till we fix issues in IPSCCP, this patch stands reverted. The whole intention of this patch was to handle a subset of cases for AMDGPU backend, specifically, needed for https://reviews.llvm.org/D99347.

This doesn't necessarily mean there's a problem with IPSCCP. What is the failing IR?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98884