SuccessChanges

Summary

  1. [XCOFF] emit .rename for .lcomm when necessary (details)
  2. [GlobalISel][CallLowering] Look through call parameters for flags (details)
Commit f48eced390dcda54766e1c510af10bbcbaebcd7e by jasonliu
[XCOFF] emit .rename for .lcomm when necessary

Summary:

This is a follow up for D82481. For .lcomm directive, although it's
not necessary to have .rename emitted, it's still desirable to do
it so that we do not see internal 'Rename..' gets print out in
symbol table. And we could have consistent naming between TC entry
and .lcomm. And also have consistent naming between IR and final
object file.

Reviewed By: hubert.reinterpretcast

Differential Revision: https://reviews.llvm.org/D86075
The file was modifiedllvm/lib/Target/PowerPC/PPCAsmPrinter.cpp (diff)
The file was modifiedllvm/test/CodeGen/PowerPC/aix-xcoff-symbol-rename.ll (diff)
The file was modifiedllvm/lib/MC/MCAsmStreamer.cpp (diff)
Commit 224a8c639eeb36b7a5ac6f8a50295f9ee2cb2518 by Jessica Paquette
[GlobalISel][CallLowering] Look through call parameters for flags

We weren't looking through the parameters on calls at all.

E.g., say you had

```
declare i32 @zext(i32 zeroext %x)

...
%y = call i32 @zext(i32 %something)
...

```

At the point of the call, we wouldn't know that the %something should have the
zeroext attribute.

This sets flags in about the same way as
TargetLoweringBase::ArgListEntry::setAttributes.

Differential Revision: https://reviews.llvm.org/D86125
The file was modifiedllvm/test/CodeGen/AArch64/GlobalISel/swiftself.ll (diff)
The file was modifiedllvm/test/CodeGen/AMDGPU/GlobalISel/irtranslator-call-sret.ll (diff)
The file was modifiedllvm/test/CodeGen/AArch64/GlobalISel/call-translator.ll (diff)
The file was modifiedllvm/include/llvm/CodeGen/GlobalISel/CallLowering.h (diff)
The file was modifiedllvm/lib/CodeGen/GlobalISel/CallLowering.cpp (diff)
The file was modifiedllvm/test/CodeGen/AArch64/GlobalISel/swifterror.ll (diff)