[PATCH] Add support for ARM modified immediate syntax

[Renato Golin]

On 30 July 2013 10:35, Mihail Popa <mihail.popa at gmail.com> wrote:

> I think a good principle of taking decisions is comparing the scope of a
> change to the scope of it's potential impact.
> While with 1 the scopes are similar (assembler-only functionality confined
> to specific instructions), solution 2 exhibits
> a pretty large imbalance in this regard (the scope of the change is
> assembler-only functionality specific to a few instructions,
> the scope of the potential impact is any compiled application).
>

Hi Mihal,

I'll take your paragraph as is, not related to this issue...

Your principle is to take the shortest, least harmful path in the short
term. But if that will take you down the long term path (again) of having
to re-factor the code later on (as Jim mentioned), than it's better to
re-factor it now, with a simpler, less buggy and more cohesive code.

My view is that the Occam's razor works better when you take into account
short AND long term goals. Failure to understand this is probably why the
world governments, corporations and markets are failing right now, and it's
not different with code. The sooner you fix things at the source, planning
also for long-term behaviour, the easier it'll be to maintain structure
amidst change.

cheers,
--renato
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130730/578e6474/attachment.html>