[PATCH] D102726: [IR][AutoUpgrade] Drop alignment from non-pointer parameters and returns
Duncan P. N. Exon Smith via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 20 12:47:58 PDT 2021
dexonsmith added a comment.
In D102726#2769856 <https://reviews.llvm.org/D102726#2769856>, @steven_wu wrote:
> In D102726#2769821 <https://reviews.llvm.org/D102726#2769821>, @dexonsmith wrote:
>
>> Can you include the revert to attributes-3.3.ll.bc as part of this?
>
> I don't think that matters too much
I think it matters. With the original file in place, it's clear that we actually upgrade correctly all the attributes that were in the 3.3 test. Without it, it's unclear.
> I am not sure if I revert a binary file will add a duplication to git history.
Every object in Git is referenced by the hash of its content. Older commits (before @jdoerfert's patch) will point at the same hash for attributes-3.3.ll.bc as the new commits after it gets reverted back. There's no duplication here.
> The case in 3.3 version is kind of covered by newly added test case except created by different llvm version.
Yeah, kind of. But not entirely.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D102726/new/
https://reviews.llvm.org/D102726
More information about the llvm-commits
mailing list