[clang][OpenMP] Add 'allocator' modifier for 'allocate' clause. by ddpagan · Pull Request #114883 · llvm/llvm-project (original) (raw)

Conversation

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

The 'allocator' modifier is now accepted in the 'allocate' clause. Added LIT tests covering codegen, PCH, template handling, and serialization for 'allocator' modifier.

Added support for allocator-modifier to release notes.

Testing

• New allocate modifier LIT tests.
• OpenMP LIT tests.
• check-all
• relevant sollve_vv test cases tests/5.2/scope/test_scope_allocate_construct.c

Comment on lines 2258 to 2259

Comment on lines 4542 to 4543

• Added '{' and '}' to else with single statement per coding standards.
• BalancedDelimiterTracker() definition moved to where it's used.

qiaojbao pushed a commit to GPUOpen-Drivers/llvm-project that referenced this pull request

Dec 4, 2024

…fa63deffa

Local branch amd-gfx 359fa63 Merged main:ce112a7f44ca0776d1192f6183a33e0c9f69df53 into amd-gfx:a12fc4896927 Remote branch main 435e584 [clang][OpenMP] Add allocator modifier for allocate clause. (llvm#114883)

Change-Id: I38854570a352c5dfa5b9acc7ddd7af040b446373