Started 1 yr 4 mo ago
Took 2 hr 25 min on green-dragon-03

Success Build rL:366582 - C:366546 - #63357 (Jul 19, 2019 7:17:55 AM)

  • : 366582
  • : 366546
  1. AMDGPU/GlobalISel: Rewrite lowerFormalArguments

    This should now handle everything except structs passed as multiple

    I think most of the packing logic should be handled by
    handleAssignments, but I'm unclear on what the contract is for
    multiple registers. This is copying how x86 handles this.

    This does change the behavior of the test_sgpr_alignment0 amdgpu_vs
    test. I don't think shader arguments should try to follow the
    alignment, and registers need to be repacked. I also don't think it
    matters, since I think the pointers are packed to the beginning of the
    argument list anyway. (detail/ViewSVN)
    by arsenm
  2. AMDGPU: Decompose all values to 32-bit pieces for calling conventions

    This is the more natural lowering, and presents more opportunities to
    reduce 64-bit ops to 32-bit.

    This should also help avoid issues graphics shaders have had with
    64-bit values, and simplify argument lowering in globalisel. (detail/ViewSVN)
    by arsenm
  3. gn build: Set +x on (detail/ViewSVN)
    by nico
  4. DAG: Handle dbg_value for arguments split into multiple subregs

    This was handled previously for arguments split due to not fitting in
    an MVT. This was dropping the register for argument registers split
    due to TLI::getRegisterTypeForCallingConv. (detail/ViewSVN)
    by arsenm

Started by an SCM change (4 times)

This run spent:

  • 36 min waiting;
  • 2 hr 25 min build duration;
  • 3 hr 2 min total from scheduled to completion.
LLVM/Clang Warnings: 1 warning.
Test Result (no failures)