lebedev.ri marked 2 inline comments as done.
lebedev.ri added inline comments.


================
Comment at: clang/test/CodeGen/assume-aligned-and-alloc-align-attributes.c:12
 // CHECK-NEXT:    [[MASKCOND:%.*]] = icmp eq i64 [[MASKEDPTR]], 0
 // CHECK-NEXT:    call void @llvm.assume(i1 [[MASKCOND]])
 // CHECK-NEXT:    ret i8* [[CALL]]
----------------
jdoerfert wrote:
> Why would we still emit the llvm.assume here?
Because it is for the second/other attribute - 
`__attribute__((alloc_align(2)))`.
This is **potentially** fixed by the next patch - D73006.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73005



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to