[PATCH] D111293: [CFE][Codegen][In-progress] Remove CodeGenFunction::InitTempAlloca()

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 7 14:35:00 PDT 2021


rjmccall added a comment.

We should take the opportunity to add a test here for the ObjC case.  It should be testable with something like this, where you should see the store inside of the loop instead of once in the prologue:

  // RUN: %clang_cc1 -triple x86_64-unknown-freebsd -S -emit-llvm -fobjc-runtime=gnustep-2.0 -o - %s | FileCheck %s
  
  typedef struct {
    int x[12];
  } Big;
  
  @protocol P
  - (Big) foo;
  @end
  
  void test(id<P> p) {
    for (;;) {
      Big big = [p foo];
    }
  }


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111293



More information about the cfe-commits mailing list