Details
Details
- Reviewers
- rampitec 
Diff Detail
Diff Detail
Event Timeline
| lib/Target/AMDGPU/AMDGPUISelLowering.cpp | ||
|---|---|---|
| 3457 | You need to check bitwidth here. 0x3118 is a valid float 1.7612E-41. I assume same is true for double. | |
| Paths 
 |  Differential  D50705  
AMDGPU: Address todo for handling 1/(2 pi) ClosedPublic Authored by arsenm on Aug 14 2018, 7:41 AM. 
Details 
 
Diff Detail Event TimelineHerald added subscribers: t-tye, tpr, dstuttard and 4 others.  ·  View Herald TranscriptAug 14 2018, 7:41 AM 
 This revision is now accepted and ready to land.Aug 14 2018, 11:52 PM 
Revision Contents 
 
Diff 160583 lib/Target/AMDGPU/AMDGPUISelLowering.h
 lib/Target/AMDGPU/AMDGPUISelLowering.cpp
 lib/Target/AMDGPU/AMDGPUSubtarget.h
 lib/Target/AMDGPU/AMDGPUSubtarget.cpp
 test/CodeGen/AMDGPU/fneg-combines.ll
 test/CodeGen/AMDGPU/fneg-combines.si.ll
 
 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
You need to check bitwidth here. 0x3118 is a valid float 1.7612E-41. I assume same is true for double.