[all-commits] [llvm/llvm-project] 276c8d: [clang][codegen] Refactor argument loading in func...

[darkbuck via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 276c8dde0b58cfe29035448a27e16eff9fcf2a5a
      https://github.com/llvm/llvm-project/commit/276c8dde0b58cfe29035448a27e16eff9fcf2a5a
  Author: Michael Liao <michael.hliao at gmail.com>
  Date:   2020-05-05 (Tue, 05 May 2020)

  Changed paths:
    M clang/lib/CodeGen/CGCall.cpp
    M clang/lib/CodeGen/CodeGenFunction.h

  Log Message:
  -----------
  [clang][codegen] Refactor argument loading in function prolog. NFC.

Summary:
- Skip copying function arguments and unnecessary casting by using them
  directly.

Reviewers: rjmccall, kerbowa, yaxunl

Subscribers: cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D79394


  Commit: 9142c0b46bfea13d9348ab3d1d706a10ad9e5c8e
      https://github.com/llvm/llvm-project/commit/9142c0b46bfea13d9348ab3d1d706a10ad9e5c8e
  Author: Michael Liao <michael.hliao at gmail.com>
  Date:   2020-05-05 (Tue, 05 May 2020)

  Changed paths:
    M clang/lib/CodeGen/CGCall.cpp
    M clang/test/CodeGenCUDA/amdgpu-kernel-arg-pointer-type.cu

  Log Message:
  -----------
  [clang][codegen] Hoist parameter attribute setting in function prolog.

Summary:
- If the coerced type is still a pointer, it should be set with proper
  parameter attributes, such as `noalias`, `nonnull`, and etc. Hoist
  that (pointer) parameter attribute setting so that the coerced pointer
  parameter could be marked properly.

Depends on D79394

Reviewers: rjmccall, kerbowa, yaxunl

Subscribers: jvesely, nhaehnle, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D79395


Compare: https://github.com/llvm/llvm-project/compare/86e50af72d0a...9142c0b46bfe