Re: [jamboost] Re: [BBv2] alignment feature and msvc assembler support

From: Vladimir Prus (ghost_at_[hidden])
Date: 2005-09-19 04:15:28

Hi Reece,

> I have added this in the msvc-asm.diff patch as well as moving the
> safe-asynch-exceptions feature to this patch. I also fixed a few things
> like adding .ASM in init ( ... ).
>
> The dependancies are now:
> 1. empty-optional.diff (from Alexey Pakhunov)

Committed now.

> 2. alignment.diff

I've some concerns. 

1. What are use cases for <alignment>? If that's for specific layout for 
specific structures, you've better off using pragmas, no? In that case, 
there's little point in feature.

2. Do you really ever want to build the same projects with several alignments, 
and have several build directories, one for each alignment? Does not seem 
likely for me. And if not, the feature should be incidental.

3. If <safe-asynch-exceptions> is so specific to msvc, why adding a feature. 
Won't <cxxflags>/safeseh work just as well?

4. Note the "link-incompatible" attribute of feature is not tested, never 
worked nice, and probably does not work at all now. I suggest avoiding it, 
and will remove it from the docs soon.

> 3. msvc-asm.diff

Is ia64 variable used anywhere?

- Volodya

-- 
Vladimir Prus
http://vladimir_prus.blogspot.com
Boost.Build V2: http://boost.org/boost-build2