[PATCH] D101806: [TargetLowering] Only inspect attributes in the arguments for ArgListEntry
Ulrich Weigand via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 27 09:42:24 PDT 2021
uweigand added a comment.
I looks like this patch, when committed as 1c7f32334d4becc725b9025fd32291a0e5729acd <https://reviews.llvm.org/rG1c7f32334d4becc725b9025fd32291a0e5729acd>, caused two additional build bot failures on s390x:
https://lab.llvm.org/buildbot/#/builders/94/builds/3829
FAIL: libFuzzer:: msan.test
FAIL: libFuzzer:: swap-cmp.test
This was unfortunately hidden by other failures at the time, so I didn't notice earlier.
I see that https://reviews.llvm.org/D102667 was an attempt to fix problems with this patch for msan, but it addressed only the zeroext attribute. On s390x we also often require the signext attribute, in particular for a plain "int" function argument or return value. I have not looked into this in detail, but I see in msan_interface_internal.h quite a number of routines using "int", so it wouldn't surprise me if this could explain the regression ...
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D101806/new/
https://reviews.llvm.org/D101806
More information about the llvm-commits
mailing list