Bug fix in Tblgen

Malul, Elior elior.malul at intel.com
Sun Apr 14 03:16:28 PDT 2013


Hi Jacob, attached is the new patch, in which the value propagation is done without adding it iterator artificially to the record.
Will appreciate any reviews and comments.
Thx, Elior

-----Original Message-----
From: Jakob Stoklund Olesen [mailto:stoklund at 2pi.dk] 
Sent: Tuesday, April 09, 2013 19:41
To: Malul, Elior
Cc: llvm-commits at cs.uiuc.edu; Sean Silva
Subject: Re: Bug fix in Tblgen


On Apr 7, 2013, at 3:53 AM, "Malul, Elior" <elior.malul at intel.com> wrote:

> I think what you say make sense, since after all what I did is a solution to the problem, not to the problem itself.
> The downside is that we will need to make another 'resolve mechanism' (since the current resolve mechanism only resolves fields).
> Are you OK with that?

Yes.

I think the 'foreach' iteration variables should be lexically scoped, and not dynamically scoped like the 'let' bindings.

This would mean that you recognize them as iteration variables while parsing the defs inside a foreach loop, instead of back-substituting after parsing as it is currently done.

BTW, the whole Init::resolveReferencesTo() system is pretty slow, walking the entire expression tree for every variable substitution. If you can think of a faster way, that would be great.

/jakob

---------------------------------------------------------------------
Intel Israel (74) Limited

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.





More information about the llvm-commits mailing list