[PATCH] D26769: [IR] Remove the DIExpression field from DIGlobalVariable.
Adrian Prantl via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 16 11:53:22 PST 2016
FYI:
On Dec 16, 2016, at 11:39 AM, Adrian Prantl via llvm-commits <llvm-commits at lists.llvm.org> wrote:
Author: adrian
Date: Fri Dec 16 13:39:01 2016
New Revision: 289982
URL: http://llvm.org/viewvc/llvm-project?rev=289982&view=rev
Log:
Revert "[IR] Remove the DIExpression field from DIGlobalVariable."
This reverts commit 289920 (again).
I forgot to implement a Bitcode upgrade for the case where a DIGlobalVariable
has not DIExpression. Unfortunately it is not possible to safely upgrade
these variables without adding a flag to the bitcode record indicating which
version they are.
My plan of record is to roll the planned follow-up patch that adds a
unit: field to DIGlobalVariable into this patch before recomitting.
This way we only need one Bitcode upgrade for both changes (with a
version flag in the bitcode record to safely distinguish the record
formats).
Sorry for the churn!
More information about the llvm-commits
mailing list