Hi James

On 19/12/18 3:40 PM, James Greenhalgh wrote:
> On Fri, Dec 14, 2018 at 10:09:03AM -0600, Sudakshina Das wrote:
> 
> <snip>
> 
>> I have updated the patch according to our discussions offline.
>> The md pattern is now split into 4 patterns and i have added a new
>> test for the setjmp case along with some comments where missing.
> 
> This is OK for trunk.
> 

Thanks for the approvals. With this my series is ready to go in trunk. I 
will wait for Sam's options patch to go in trunk before I commit mine.

Thanks
Sudi

> Thanks,
> James
> 
>> *** gcc/ChangeLog ***
>>
>> 2018-xx-xx  Sudakshina Das  <sudi....@arm.com>
>>          Ramana Radhakrishnan  <ramana.radhakrish...@arm.com>
>>
>>      * config.gcc (aarch64*-*-*): Add aarch64-bti-insert.o.
>>      * gcc/config/aarch64/aarch64.h: Update comment for
>>      TRAMPOLINE_SIZE.
>>      * config/aarch64/aarch64.c (aarch64_asm_trampoline_template):
>>      Update if bti is enabled.
>>      * config/aarch64/aarch64-bti-insert.c: New file.
>>      * config/aarch64/aarch64-passes.def (INSERT_PASS_BEFORE): Insert
>>      bti pass.
>>      * config/aarch64/aarch64-protos.h (make_pass_insert_bti):
>>      Declare the new bti pass.
>>      * config/aarch64/aarch64.md (unspecv): Add UNSPECV_BTI_NOARG,
>>      UNSPECV_BTI_C, UNSPECV_BTI_J and UNSPECV_BTI_JC.
>>      (bti_noarg, bti_j, bti_c, bti_jc): New define_insns.
>>      * config/aarch64/t-aarch64: Add rule for aarch64-bti-insert.o.
>>
>> *** gcc/testsuite/ChangeLog ***
>>
>> 2018-xx-xx  Sudakshina Das  <sudi....@arm.com>
>>
>>      * gcc.target/aarch64/bti-1.c: New test.
>>      * gcc.target/aarch64/bti-2.c: New test.
>>      * gcc.target/aarch64/bti-3.c: New test.
>>      * lib/target-supports.exp
>>      (check_effective_target_aarch64_bti_hw): Add new check for
>>      BTI hw.
>>
>> Thanks
>> Sudi

Reply via email to