SuccessChanges

Summary

  1. [NFC] Style cleanup (details)
  2. [Utils][Fix] Minor test result change (details)
  3. [X86] Slightly improve our attempted error recovery for 64-bit -mno-sse2 (details)
  4. [Attributor] Function signature rewrite infrastructure (details)
  5. [Utils] Deal with occasionally deleted functions (details)
  6. [Attributor] Suppress unused warnings when assertions are disabled. NFC (details)
Commit a36a89dcdcfc70de09693e62939009aaa0e52dd5 by shengchen.kan
 [NFC] Style cleanup
1. make function Is16BitMemOperand static 2. Use Doxygen features in
comment 3. Rename functions to make them start with a lower case letter
The file was modifiedllvm/lib/Target/X86/MCTargetDesc/X86MCCodeEmitter.cpp
Commit 4a6413cd0a8d6bf24a2c67800179e3990ca50a84 by johannes
[Utils][Fix] Minor test result change
The file was modifiedllvm/test/tools/UpdateTestChecks/update_test_checks/Inputs/argument_name_reuse.ll.expected
Commit e898ba2d151d621dcfc35828aad6fcded5a554e8 by craig.topper
[X86] Slightly improve our attempted error recovery for 64-bit -mno-sse2
in LowerCallResult to use FP1 if there are two return values.
If the return value is a struct of 2 doubles we need two return
registers.
If SSE2 is disabled we can't return in XMM registers like the ABI says.
After logging an error we attempt to recover by using FP0 instead of an
XMM register. But if the return needs two registers, we may have already
used FP0. So if the register we were supposed to copy to is XMM1, copy
to FP1 in the recovery instead.
This seems to fix the assertion/crash in PR44413.
The file was modifiedllvm/lib/Target/X86/X86ISelLowering.cpp
Commit 751336340dabc58f178a24f7b169f33366316f70 by johannes
[Attributor] Function signature rewrite infrastructure
As part of the Attributor manifest we want to change the signature of
functions. This patch introduces a fairly generic interface to do so. As
a first, very simple, use case, we remove unused arguments. A second use
case, pointer privatization, will be committed with this patch as well.
A lot of the code and ideas are taken from argument promotion and we run
all argument promotion tests through this framework as well.
Reviewed By: uenoku
Differential Revision: https://reviews.llvm.org/D68765
The file was modifiedllvm/include/llvm/Transforms/IPO/Attributor.h
The file was modifiedllvm/test/Transforms/Attributor/nonnull.ll
The file was modifiedllvm/test/Transforms/Attributor/ArgumentPromotion/pr27568.ll
The file was modifiedllvm/lib/Transforms/IPO/Attributor.cpp
The file was modifiedllvm/test/Transforms/Attributor/liveness.ll
The file was modifiedllvm/test/Transforms/Attributor/noalias.ll
The file was modifiedllvm/test/Transforms/Attributor/ArgumentPromotion/control-flow.ll
The file was modifiedllvm/test/Transforms/Attributor/ArgumentPromotion/dbg.ll
The file was modifiedllvm/test/Transforms/Attributor/ArgumentPromotion/reserve-tbaa.ll
The file was modifiedllvm/test/Transforms/Attributor/ArgumentPromotion/nonzero-address-spaces.ll
Commit a6c59e0792edc46df937b338fe0e68d00cabf90b by johannes
[Utils] Deal with occasionally deleted functions
When functions exist for some but not all run lines we need to be
careful when selecting the prefix. So far, a common prefix was
potentially chosen as there was never a "conflict" that would have
caused otherwise. With this patch we avoid common prefixes if they are
used by run lines that do not emit the function.
Reviewed By: lebedev.ri
Differential Revision: https://reviews.llvm.org/D68850
The file was modifiedllvm/utils/UpdateTestChecks/common.py
The file was addedllvm/test/tools/UpdateTestChecks/update_test_checks/Inputs/sometimes_deleted_function.ll.expected
The file was addedllvm/test/tools/UpdateTestChecks/update_test_checks/sometimes_deleted_function.test
The file was addedllvm/test/tools/UpdateTestChecks/update_test_checks/Inputs/sometimes_deleted_function.ll
Commit 4f82af81a04d711721300f6ca32f402f2ea6faf4 by ibiryukov
[Attributor] Suppress unused warnings when assertions are disabled. NFC
The file was modifiedllvm/lib/Transforms/IPO/Attributor.cpp