[PATCH] D86272: [DebugInfo] Fix DwarfExpression::addConstantFP for float on big-endian

Sourabh Singh Tomar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 20 03:29:28 PDT 2020


SouraVX added a comment.

Thanks @bjope for catching this bug! I do not have big-endian machine so it must've got un-noticed(test case for 4 bytes float was written and tested/verified on x86 machine). 
But I got your point, I'll see if I can add a hand written test case to address this situation.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86272/new/

https://reviews.llvm.org/D86272



More information about the llvm-commits mailing list