Skip to content
  • Matt Arsenault's avatar
    Consolidate internal denormal flushing controls · a4451d88
    Matt Arsenault authored
    Currently there are 4 different mechanisms for controlling denormal
    flushing behavior, and about as many equivalent frontend controls.
    
    - AMDGPU uses the fp32-denormals and fp64-f16-denormals subtarget features
    - NVPTX uses the nvptx-f32ftz attribute
    - ARM directly uses the denormal-fp-math attribute
    - Other targets indirectly use denormal-fp-math in one DAGCombine
    - cl-denorms-are-zero has a corresponding denorms-are-zero attribute
    
    AMDGPU wants a distinct control for f32 flushing from f16/f64, and as
    far as I can tell the same is true for NVPTX (based on the attribute
    name).
    
    Work on consolidating these into the denormal-fp-math attribute, and a
    new type specific denormal-fp-math-f32 variant. Only ARM seems to
    support the two different flush modes, so this is overkill for the
    other use cases. Ideally we would error on the unsupported
    positive-zero mode on other targets from somewhere.
    
    Move the logic for selecting the flush mode into the compiler driver,
    instead of handling it in cc1. denormal-fp-math/denormal-fp-math-f32
    are now both cc1 flags, but denormal-fp-math-f32 is not yet exposed as
    a user flag.
    
    -cl-denorms-are-zero, -fcuda-flush-denormals-to-zero and
    -fno-cuda-flush-denormals-to-zero will be mapped to
    -fp-denormal-math-f32=ieee or preserve-sign rather than the old
    attributes.
    
    Stop emitting the denorms-are-zero attribute for the OpenCL flag. It
    has no in-tree users. The meaning would also be target dependent, such
    as the AMDGPU choice to treat this as only meaning allow flushing of
    f32 and not f16 or f64. The naming is also potentially confusing,
    since DAZ in other contexts refers to instructions implicitly treating
    input denormals as zero, not necessarily flushing output denormals to
    zero.
    
    This also does not attempt to change the behavior for the current
    attribute. The LangRef now states that the default is ieee behavior,
    but this is inaccurate for the current implementation. The clang
    handling is slightly hacky to avoid touching the existing
    denormal-fp-math uses. Fixing this will be left for a future patch.
    
    AMDGPU is still using the subtarget feature to control the denormal
    mode, but the new attribute are now emitted. A future change will
    switch this and remove the subtarget features.
    a4451d88
Loading