(S) RFR: 8077648: ARM: BREAKPOINT is wrong for thumb

[Dean Long]

Looks good.

dl

On 1/12/2016 6:27 PM, David Holmes wrote:
> On 12/01/2016 8:03 PM, Edward Nevill wrote:
>> On Tue, 2016-01-12 at 15:56 +1000, David Holmes wrote:
>>> On 12/01/2016 3:31 PM, Dean Long wrote:
>>>> How about moving it to "globalDefinitions_arm.hpp"?  You would just 
>>>> need
>>>> to do #undef BREAKPOINT first.
>>>
>>> It would also need, technically, to be conditional upon using gcc.
>>>
>>> I also don't know if the aarch32 port would be using this. ??
>>
>> cc: aarch32-port-dev
>>
>> The aarch32 currently does not define 'ARM32' so this code is not 
>> being included in the current port.
>>
>> The aarch32 port does define 'ARM' so any code that is #ifdef ARM is 
>> included in the aarch32 port.
>>
>> So, moving this code to globalDefinitions_arm.hpp will have no impact 
>> on the aarch32 port.
>
> Thanks Ed. Okay updated webrev in place:
>
> http://cr.openjdk.java.net/~dholmes/8077648/webrev/
>
> All ARM related code removed.
>
> Thanks,
> David
> -----
>
>> All the best,
>> Ed.
>>
>>