[PATCH] D73664: [PowerPC][Future] Add Support For Functions That Do Not Use A TOC.

[Nemanja Ivanovic via Phabricator via llvm-commits]

nemanjai added a comment.

In D73664#1974913 <https://reviews.llvm.org/D73664#1974913>, @nathanchance wrote:

> @nemanjai thank you for that fix! Unfortunately, I am still seeing some errors with the `modules` target:
>
>   $ make -j$(nproc) -s AR=llvm-ar ARCH=powerpc CC=clang CROSS_COMPILE=powerpc64le-linux-gnu- NM=llvm-nm O=out/ppc64le OBJCOPY=llvm-objcopy OBJDUMP=llvm-objdump OBJSIZE=llvm-size READELF=llvm-readelf STRIP=llvm-strip distclean powernv_defconfig modules
>   ...
>   /tmp/int4-4420da.s: Assembler messages:
>   /tmp/int4-4420da.s:176: Error: can't resolve `.TOC.' {*UND* section} - `.Lfunc_gep1' {*UND* section}
>   clang-11: error: assembler command failed with exit code 1 (use -v to see invocation)
>   ...
>
>
> Sorry I did not catch this initially, we were just building the kernel image before, I did not realize that there was going to be a separate set of issues with modules :/ our CI only builds the kernel image due to time restrictions but I build everything locally, hence how I did not catch that sooner.
>
> The command above is just for `modules` for easy triage. You can also just build one translation unit like `lib/raid6/int4.o`, that one errors for me. I build everything locally with the `all` target.


This is also a regression after this patch or something that was broken before?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73664