We're ok with env field being blank, so this doesn't accomplish what we need

@thakis thanks for reverting that, sorry I didn't see your request sooner!

Updating driver cmake to correctly build shared library

Removing trailing whitespace

Fixing header typo

Switching pointers to references, adding namespace comment

Bug fix

Moving classes into clang namespace, more specific name for config class

Abandoned in favor of combined patch: https://reviews.llvm.org/D129873

Abandoned in favor of combined patch: https://reviews.llvm.org/D129873

Abandoned in favor of combined patch: https://reviews.llvm.org/D129873

Abandoned in favor of combined patch: https://reviews.llvm.org/D129873

@yaxunl Are you recommending I combine all 4 patches down into 1 patch? Or combine a subset of patches?

Adding clang-format

Adding clang-format

Adding clang-format

Switching test

Adding test

Updating commit message

Addressing clang-format comments

Address clang-format comment

Updating commit message

Switching to use new getTotalNumVGPRs from AMDGPUBaseInfo

Updating commit message

Fix small formatting error

Removing llvm namespace from function call and addressing clang-format issues

Adding revision info to commit message

Clarifying description

Fixing MCSymbol const declaration formatting

Updating commit message

Reverting code changes but keeping updated formatting to fix build error (see previous comment)

Reformatting (clang-format suggestion)

Removing accidental print statement and fixing formatting

Removing extra newline

Updating description to match hardware description, adding appropriate checks

Adding checks for wavefront size and multiple of 8

@rampitec @arsenm If I'm understanding correctly, moving const to the left of "MCSymbol *"' declares the object as const instead of the pointer. In this case, we get a compilation error on the following line:

Fixing whitespace

Moved instead of removed const qualifiers

Add check for shared_vgpr_count exceeding next_free_vgpr

Removing const from MCSymbol variable and creating fall through case for getTotalNumVgprs()

Addressing comments from kerbowa and lint tool.