[PATCH] D87304: [AttributeFuncs] Consider `align` in `typeIncompatible`

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added inline comments.


================
Comment at: llvm/test/Bitcode/attributes-3.3.ll:105
+define void @f17(i8* align 4 %0)
+; CHECK: define void @f17(i8* align 4 %0)
 {
----------------
dexonsmith wrote:
> jdoerfert wrote:
> > fhahn wrote:
> > > just to clarify, `attributes-3.3.ll.bc` needs to be updated, because it contains bitcode that defines a function with align on a non-pointer argument?
> > it was updated because of that, yes.
> Hmm, I'm just seeing this now -- but attributes-3.3.ll.bc should not have been changed. Can that part of the change be reverted? Instead, the bitcode reader should have been adjusted to auto-upgrade the bitcode and strip the align attribute where appropriate.
> 
> Note that eventually the problem hit in production and @steven_wu added the bitcode upgrade in https://reviews.llvm.org/D102201. The whole reason we have these bitcode tests is to catch upgrade issues before commit.
With D102201 landed we can revert attributes-3.3.ll.bc. It makes sense to do it that way.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87304