[all-commits] [llvm/llvm-project] 560519: clang: Remove dest LangAS argument from performAdd...

Matt Arsenault via All-commits all-commits at lists.llvm.org
Wed May 7 22:55:12 PDT 2025


  Branch: refs/heads/users/arsenm/clang/remove-dest-lang-as-from-performAddrSpaceCast
  Home:   https://github.com/llvm/llvm-project
  Commit: 56051957434e51e3b4108419348420bd54be8506
      https://github.com/llvm/llvm-project/commit/56051957434e51e3b4108419348420bd54be8506
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2025-05-08 (Thu, 08 May 2025)

  Changed paths:
    M clang/lib/CodeGen/CGAtomic.cpp
    M clang/lib/CodeGen/CGBuiltin.cpp
    M clang/lib/CodeGen/CGCall.cpp
    M clang/lib/CodeGen/CGClass.cpp
    M clang/lib/CodeGen/CGDecl.cpp
    M clang/lib/CodeGen/CGException.cpp
    M clang/lib/CodeGen/CGExpr.cpp
    M clang/lib/CodeGen/CGExprCXX.cpp
    M clang/lib/CodeGen/CGExprConstant.cpp
    M clang/lib/CodeGen/CGExprScalar.cpp
    M clang/lib/CodeGen/CodeGenModule.cpp
    M clang/lib/CodeGen/TargetInfo.cpp
    M clang/lib/CodeGen/TargetInfo.h

  Log Message:
  -----------
  clang: Remove dest LangAS argument from performAddrSpaceCast

It isn't used and is redundant with the result pointer type argument.
A more reasonable API would only have LangAS parameters, or IR parameters,
not both. Not all values have a meaningful value for this. I'm also
not sure why we have this at all, it's not overridden by any targets and
further simplification is possible.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list