This revision was automatically updated to reflect the committed changes.
Closed by commit rL317627: Change noplt.c to work for non-x86 targets.
(authored by tmsriram).
Changed prior to commit:
https://reviews.llvm.org/D39759?vs=121977&id=121987#toc
Repository:
rL LLVM
https://reviews.llvm.
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
Please search for `declare {{.*}}i32 @foo`, and then commit. Otherwise this
could match `call i32 @foo` or something like that.
https://reviews.llvm.org/D39759
__
tmsriram added a comment.
More information here. This breaks non-x86 builds like PPC because:
/home/buildbots/ppc64be-clang-test/clang-ppc64be/llvm/tools/clang/test/CodeGen/noplt.c:4:22:
error: expected string not found in input
// CHECK-NOPLT-NEXT: declare i32 @foo
^
:14:1: note: scanning
tmsriram created this revision.
https://reviews.llvm.org/D39079 breaks noplt.c for non-x86 targets. Remove x86
specific code from noplt.c
https://reviews.llvm.org/D39759
Files:
test/CodeGen/noplt.c
Index: test/CodeGen/noplt.c
===
