[PATCH] D156539: [Clang][CodeGen] `__builtin_alloca`s should care about address spaces too

Matt Arsenault via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 28 09:15:36 PDT 2023


arsenm added inline comments.


================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:3540
+      return RValue::get(Builder.CreateAddrSpaceCast(AI, CGM.Int8PtrTy));
+    else
+      return RValue::get(AI);
----------------
No return after else


================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:3558
+      return RValue::get(Builder.CreateAddrSpaceCast(AI, CGM.Int8PtrTy));
+    else
+      return RValue::get(AI);
----------------
No return after else


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D156539/new/

https://reviews.llvm.org/D156539



More information about the cfe-commits mailing list